-
Notifications
You must be signed in to change notification settings - Fork 35
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
Comment out manual InvMEGAN diagnostic creation code #281
Comments
Thanks for reporting this @eamarais. I've added it to our to-do list for an upcoming release (GEOS-Chem 14.4.x or 14.5.0). |
Hi @msulprizio and @eamarais, thanks for bringing this up. I wonder if we could figure out a way to leave the Also note, in PR #2372 we are implementing a fix to turn off the MEGAN extension for all fullchem simulations except fullchem_benchmark. This has the added benefit of disabling the InvMEGAN diagnostic output. See this comment. |
Hi @yantosca, these should be output as long as these are specified in the HEMCO_Diagn.rc file. |
@eamarais: Maybe an easy fix is to delete those lines from the HEMCO_Diagn.rc at rundir creation for all fullchem simulations except benchmark. I can look into this. |
@yantosca, I don't quite follow your suggestion, as my recommendation is that it makes sense to no longer have these Diagn_Create lines in the code. These are no longer needed, as the request to output the InvMEGAN* variables can be specified by users in HEMCO_Diagn.rc and users can output these when they want, rather than having these output by default. |
@eamarais: Oh yes, now I see. Brain freeze. |
This commit addresses the issue in geoschem/HEMCO #281. src/Extensions/hcox_megan_mod.F90 - Removed the calls to Diagn_Create for InvMEGAN diagnostics. These can be activated by adding the proper entries to the HEMCO_Diagn.rc file. CHANGELOG.md - Updated accordingly Signed-off-by: Bob Yantosca <yantosca@seas.harvard.edu>
Your name
Eloise Marais
Your affiliation
UCL, UK
What happened? What did you expect to happen?
This is only sort of a bug. It's a request to comment out unnecessary code that outputs lots of emissions estimates by default when these aren't wanted. Specifically, to comment out calls to Diagn_Create code for all "InvMEGAN_*" variables in the subroutine HCOX_Megan_Init in ./HEMCO/src/Extensions/hcox_megan_mod.F90. These aren't needed, as the user can request these be output from the HEMCO diagnostics file HEMCO_Diagn.rc.
I tested and confirmed that commenting out all these lines, recompiling and rerunning GCClassic with the diagnostics requested via HEMCO_Diagn.rc works.
What are the steps to reproduce the bug?
Intentionally left blank, as this is just a quick fix detailed above.
Please attach any relevant configuration and log files.
Attached a screenshot to show 1 of the 16 diagnostics currently set manually in the code.
What HEMCO version were you using?
3.0.0 and 3.6.0 (issue is in both)
What environment were you running HEMCO on?
Local cluster
What compiler and version were you using?
gcc 9.2.0
Will you be addressing this bug yourself?
Yes
In what configuration were you running HEMCO?
GCClassic
As what resolution were you running HEMCO?
multiple (4x5, 0.25x0.3125)
What meterology fields did you use?
GEOS-FP
Additional information
No response
The text was updated successfully, but these errors were encountered: