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

abort on missing files, add cmeps #12

Merged

Conversation

DeniseWorthen
Copy link
Collaborator

@DeniseWorthen DeniseWorthen commented Jun 4, 2020

Issues #12 , #10:

  • adds abort if DATM input file is missing. Abort message will be written to PET logs and default log file. This feature was tested by attempting to integrate in time past the available data input files. The model aborted as expected.

  • adds cpl_scalars for use with CMEPS

  • simplifies the setting of the run clock

  • adds dbug_flag read from nems.configure to control message printing. Setting dbug_flag = 0 will suppress message printing. Setting dbug_flag > 5 will print all messages.

  • removes additional importstate instances where possible

Verification that the DATM clock is working and advancing as expected is shown here. Black dots are the export values over 24 hours from initial time 2011-10-01-00 at a single location on the global grid. Values exported from the DATM at each timestep are show as black dots; the blue dots are the values in the 6-hourly forcing files. Only CFSR is shown.

cfsr

@DeniseWorthen
Copy link
Collaborator Author

All Regression tests pass on Orion and Hera. Logs posted.

Hera
Orion

@DeniseWorthen DeniseWorthen marked this pull request as ready for review June 4, 2020 16:59
@DeniseWorthen
Copy link
Collaborator Author

I just realized I hadn't merged my cpl_scalars branch (needed for CMEPS) after all. I will do that and re-run the regression tests.

@DeniseWorthen
Copy link
Collaborator Author

All Regression tests pass on Orion and Hera. Logs posted.

Hera
Orion

@DeniseWorthen
Copy link
Collaborator Author

@binli2337, @flampouris : This PR is ready now. Could you review? Thanks.

Copy link
Collaborator

@flampouris flampouris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @DeniseWorthen !

@DeniseWorthen DeniseWorthen merged commit a14e3bb into NOAA-EMC:develop Jun 9, 2020
@DeniseWorthen DeniseWorthen deleted the feature/abortmissingfile branch November 3, 2020 11:53
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.

4 participants