Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

create a new branch for bgc #2

Closed
hakaseh opened this issue Oct 31, 2021 · 4 comments
Closed

create a new branch for bgc #2

hakaseh opened this issue Oct 31, 2021 · 4 comments

Comments

@hakaseh
Copy link
Collaborator

hakaseh commented Oct 31, 2021

Hi @aekiss,

I'm starting to create a new branch for make_diag_table to add BGC diags. As shown below, I think you have already started and may have added all in the past, but I will try to update the repository based on this. Let me know if you have any advice.

=======================

I've set up one-field-per-file BGC outputs - see diag_table* in https://github.com/COSIMA/1deg_jra55_iaf/tree/bgc-mdppm-run/ocean and the output files in /scratch/v45/aek156/access-om2/archive/1deg_iamip_test_mdppm/output003/ocean/*bgc* - does that look OK to you?

Originally posted by @aekiss in COSIMA/1deg_jra55_iaf#11 (comment)

@hakaseh
Copy link
Collaborator Author

hakaseh commented Oct 31, 2021

@aekiss
i should add all the bgc diagnostics in diag_table_source.yaml and that's probably all I need to do, correct?
given that this is the default setup, i'll try to choose a temporal resolution reasonable for each diagnostic.

@hakaseh
Copy link
Collaborator Author

hakaseh commented Nov 1, 2021

another question. would you prefer to have a separate branch like master+bgc for make_diag_table too? If bgc diags are listed in diag_table but the model is physics-only, it will only give warning and won't force stop the run, but this will give a long list of warnings in access-om2.[err,out]. let me know if you prefer a separate branch.

@hakaseh
Copy link
Collaborator Author

hakaseh commented Nov 1, 2021

could you give me a permission to create a remote branch master+bgc? I got the following error:

(base) [hh0162@gadi-login-07 make_diag_table]$ git push origin bgc-dev-hh:master+bgc
Username for 'https://github.com': hakaseh
Password for 'https://hakaseh@github.com': 
remote: Permission to COSIMA/make_diag_table.git denied to hakaseh.
fatal: unable to access 'https://github.com/COSIMA/make_diag_table.git/': The requested URL returned error: 403

My next step is to create a control directory for master+bgc and run the 1-deg model for a year with the updated diag_table.

@hakaseh hakaseh closed this as completed Nov 9, 2021
@aekiss
Copy link
Contributor

aekiss commented Nov 11, 2021

Hi @hakase, sorry, I only just saw this. I don't really think it's necessary to put the bgc stuff in this make_diag_table repo, as it only contains an example diag_table_source.yaml that is not actually used in configurations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants