Skip to content

Conversation

@qwert2266
Copy link
Contributor

@qwert2266 qwert2266 commented Jan 25, 2024

Working on creating experiment, readme is here now and will add other code later

@qwert2266 qwert2266 changed the title Create README.md Create experiment for cpl_gray+swamp+ocn Jan 25, 2024
Before, input was the old version with no continent. Now input folders should include the necessary files
Each build directory had some remnants of building the executables previously. Removing that now, only leaving genmake_local
@jm-c
Copy link
Member

jm-c commented Jan 25, 2024

@qwert2266 Thanks for sharing your set-up, this is great. I will take a look in the coming days and will see if there is something to add or change.

jm-c added 2 commits March 7, 2024 11:44
also add ".gitignore" and update shares_code (removing CVS headers)
Also save atm & ocn multi-procs SIZE.h (suffix: .24procs, .3procs)
and add single-procs version (easier for testing).
@jm-c
Copy link
Member

jm-c commented Mar 7, 2024

@qwert2266 I did some cleaning so that I can now compile for single-proc component (easier for testing). I still have issues to run, because the input_ocn/data setting of delR is not consistent with the rest of the set-up (Nr=15 in SIZE.h, tRef,sRef setting, size of input bin files). Could you check this ?

@jm-c
Copy link
Member

jm-c commented Mar 7, 2024

And regarding number of procs to use to run atm & ocn as well as at which level to create the rank_? dir:
For now, I think it's easier to use the same convention as for the other coupled set-up, so that one can use tools/run_cpl_test to check that everything works.
In the future we could try to add new arguments to run_cpl_test so enable to change number of procs (this is already there but as hard coded parameters) and to have the rank dir one level up (but this requires also to adjust prepare_run in input_* dirs.
And for now, we would just keep a multi-proc SIZE.h version (I don't like too much the suffix ".3procs" and ".24procs" that I put there and could use soemthing else) and document this.

@qwert2266
Copy link
Contributor Author

qwert2266 commented Mar 7, 2024 via email

Updated to make consistent the necessary things to model ENSO, which are the GGL90 package and adding vertical levels to the ocean
@qwert2266
Copy link
Contributor Author

qwert2266 commented Mar 7, 2024 via email

@jm-c
Copy link
Member

jm-c commented Mar 8, 2024

@qwert2266 Thanks for the update the ocean set-up. I am going to check few other things (and may be clean-up little things) and will let you know how it goes. After that we could review the README once everything is in place.

jm-c and others added 11 commits March 10, 2024 11:07
- remove CVS Header and Name
- import changed from standard version in main code (to reduce diff with)
- reduce "numDiags" in DIAGNOSTICS_SIZE.h (but still much larger than what is
  currently used).
Assuming all rank's dir are directly put in cpl_gray+swamp+ocn/, adjust script
"prepare_run" and remove (duplicated) bin files that are available elsewhere ;
Also remove unused file "input_ocn/flat_4km.bin" and move "SST_APE_1.bin" from
input.ape/ to input/ in exp. atm_gray (a more logical place where it can be
generated with only minor changes to gendata.m there).
 -
to be used for multi-processor ocn & atm
- remove tabs
- diagnostics names are 8.c long
- correct few number of diagnostics in "fields" specification
also adjust some technical details
@jm-c jm-c merged commit a46ed92 into MITgcm:master Mar 14, 2024
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

Successfully merging this pull request may close these issues.

2 participants