search

geoschem / CAM

Community Atmosphere Model
Other
0 stars 0 forks source link

Suggested changes to cam.case_setup.py #26

Closed gold2718 closed 8 months ago

gold2718 commented 9 months ago

I added several suggestions here. I think many of them are fixes to an incomplete first pass by you.

The hard bit is likely to be with the multi-instance code. In CESM, we usually allow runtime configuration files to be specified on a per-instance basis. I added this code to the copy part of the script. However, the GEOS-Chem code that reads these files has no idea what is going on. I think you should discuss this with the AMP SEs and see if they want to push through this or to not have GEOS-Chem supported for multi-instance runs (a different but probably smaller chore).

Let me know if you have questions. --Steve

gold2718 commented 9 months ago

BTW, I do not have the ability to test this yet so it probably has bugs. (probability ~100%)

lizziel commented 8 months ago

I cherry-picked the suggested commit. Thank you @gold2718! I only had to make one minor bug fix despite your lack of testing. :)