Improve log flexibility in HEMCO #298
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ng MPI This update is relevant only for use of HEMCO read routines within MPI models. It does not impact GCHP. Signed-off-by: Lizzie Lundgren <[email protected]>
Default is 6 if not passed to ConfigInit. In GEOS-Chem this corresponds to standard output. This enables specifying destination other than stdout or HEMCO.log for HEMCO prints in models. For example, in CESM this allows printing to atm.log rather than cesm.log (stdout) by using HcoConfig%outLun in write statements. Signed-off-by: Lizzie Lundgren <[email protected]>
Signed-off-by: Lizzie Lundgren <[email protected]>
This update allows specifying which log prints in HCO_ERROR, HCO_MSG, and HCO_WARNING should be printed to if called without passed a HEMCO Err object. Signed-off-by: Lizzie Lundgren <[email protected]>
Signed-off-by: Lizzie Lundgren <[email protected]>
…assed The verb argument is an artifact of when verbose was level rather than logical. Calling HCO_MSG without the Err object should always result in a print. This supported by the currently implementation which always passeas verb=.true. when it is called. Signed-off-by: Lizzie Lundgren <[email protected]>
Signed-off-by: Lizzie Lundgren <[email protected]>
…sage Signed-off-by: Lizzie Lundgren <[email protected]>
… open This removes a check for negative LUN and only looks at logical for if file is open. CESM uses negative LUN so output should not be directed to stdout if LUN is negative. Signed-off-by: Lizzie Lundgren <[email protected]>
The CAM log is always open when HEMCO starts in CESM. Signed-off-by: Lizzie Lundgren <[email protected]>
The log file is not closed in CESM. Doing the error summary print where logfile is closed therefore results in not error summary printed in CESM. Signed-off-by: Lizzie Lundgren <[email protected]>
…_SET Previously almost all of the type members had initial values also initializaed in HCO_ERROR_SET which was redundant. Comments explaining what each field is are now added as well. Signed-off-by: Lizzie Lundgren <[email protected]>
Err%LUN is also now initialized to 6 (stdout) if LogFile is *. Signed-off-by: Lizzie Lundgren <[email protected]>
The CESM log file does not need to be opened and closed by HEMCO Signed-off-by: Lizzie Lundgren <[email protected]>
Signed-off-by: Lizzie Lundgren <[email protected]>
Signed-off-by: Lizzie Lundgren <[email protected]>
Signed-off-by: Lizzie Lundgren <[email protected]>
Signed-off-by: Lizzie Lundgren <[email protected]>
There are no remaining instances of calling HCO_Warning without passing the Err object. Signed-off-by: Lizzie Lundgren <[email protected]>
…uracy Signed-off-by: Lizzie Lundgren <[email protected]>
This specifies where to write the messages since log information in the Err object is not used. Signed-off-by: Lizzie Lundgren <[email protected]>
Signed-off-by: Lizzie Lundgren <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Name and Institution (Required)
Name: Lizzie Lundgren
Institution: Harvard University
Describe the update
Coming soon
Expected changes
This is a zero diff update for benchmarking. In CESM it enables writing all HEMCO prints to CAM log file atm.log.
Reference(s)
None
Related Github Issue
None