-
Notifications
You must be signed in to change notification settings - Fork 18
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
More informative error messages #3161
base: develop
Are you sure you want to change the base?
Conversation
…collection name in case of duplicate collection
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Subject to passing CI ...
@AgilentGCMS Please add an entry in CHANGELOG.md in the top directory. |
Looks like something went very wrong with the CI. (Unrelated to your change.) Will discuss with @mathomp4 tomorrow. |
Those are weird errors... |
Okay. I think it was just an old circleci config. I merged in the latest develop and it's happy. @AgilentGCMS Can you pull and update the CHANGELOG with an entry? |
Types of change(s)
Checklist
make tests
)Description
base/MAPL_CFIO.F90
, addedIam
to a print statement so that when a read fails we know which routine failed.gridcomps/ExtData2G/ExtDataConfig.F90
, print out the name of the duplicate collection that causes model to fail.