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

Use ESMF_LOGKIND_MULTI_ON_ERROR for logging ESMF errors #137

Merged
merged 1 commit into from
Sep 1, 2021

Conversation

LiamBindle
Copy link
Contributor

@LiamBindle LiamBindle commented Aug 25, 2021

GEOS sets cap_options%esmf_logging_mode via FLAP. Since we don't use FLAP we get the default value of ESMF_LOGKIND_NONE. IIUC this means ESMF errors dont' get logged anywhere. I'm not exactly sure whether we will ever get any ESMF errors, but ESMF_LOGKIND_MULTI_ON_ERROR seems like a more reasonable default then ESMF_LOGKIND_NONE. The "multi on error" means that each PET will dump errors to a PETXX file only if an error happens. If no error happens there are not PET* files.

I came across this option when I was troubleshooting #134. I was wondering why the ESMF_ConfigGetAttribute() call wasn't reporting a useful error message. It turns out that was something else, but I figured I should update this default setting anyway in the hopes it will aid in improving error messages.

Related

…rors are reported to PET* files (only if they occur).
@LiamBindle LiamBindle requested a review from lizziel August 25, 2021 22:41
@LiamBindle LiamBindle changed the base branch from dev to patch/13.2.1 August 30, 2021 16:43
@LiamBindle LiamBindle added this to the 13.2.1 milestone Aug 30, 2021
@LiamBindle
Copy link
Contributor Author

FYI @lizziel

@LiamBindle LiamBindle merged commit 2532d95 into patch/13.2.1 Sep 1, 2021
@yantosca yantosca deleted the feature/log-esmf-errors branch January 18, 2022 22:47
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.

1 participant