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

Fix config options for reference-year climatologies #885

Merged
merged 1 commit into from
Aug 16, 2022

Conversation

xylar
Copy link
Collaborator

@xylar xylar commented Aug 16, 2022

The start and end year for reference climatologies were getting overridden by the user's specified start and end year for normal climatologies.

(This maybe suggests that we should consider a different strategy for setting the start and end years and dates for reference climatologies that doesn't rely on overriding the user config options.)

This fixes issues where the OHC anomaly plots were exactly zero.

closes #883

@xylar xylar added the bug label Aug 16, 2022
@xylar xylar requested a review from darincomeau August 16, 2022 11:50
@xylar xylar self-assigned this Aug 16, 2022
The start and end year for reference climatologies were getting
overridden by the user's specified start and end year for normal
climatologies.

(This maybe suggests that we should consider a different
strategy for setting the start and end years and dates for
reference climatologies that doesn't rely on overriding the user
config options.)
@xylar xylar marked this pull request as ready for review August 16, 2022 12:11
@xylar xylar marked this pull request as draft August 16, 2022 12:30
@xylar
Copy link
Collaborator Author

xylar commented Aug 16, 2022

Testing

The Anvil test suite is currently running. Results are coming in here:
https://web.lcrc.anl.gov/public/e3sm/diagnostic_output/ac.xylar/analysis_testing/anvil/fix_ohc_map/

Indications are that the OHC maps are now fixed:
https://web.lcrc.anl.gov/public/e3sm/diagnostic_output/ac.xylar/analysis_testing/anvil/fix_ohc_map/main_py3.9/ocean/deltaOHC_0-10000m_main_py3.9_ANN_years0003-0008.png

Update: all analysis in the suite completed and the OHC anomaly map looks like it should in all analysis.

@xylar xylar marked this pull request as ready for review August 16, 2022 13:23
@xylar
Copy link
Collaborator Author

xylar commented Aug 16, 2022

@darincomeau, this is now ready for you to try out and review. You are correct that the bug was introduced in #873.

@xylar xylar changed the title Fix config options for reference climatologies Fix config options for reference-year climatologies Aug 16, 2022
@darincomeau
Copy link
Contributor

@xylar I reran an existing analysis with the --purge flag, and while the actual png files are populated, the thumbnails still appear blank. I see in your link above the thumbnails are populated, so I'm not sure what happened with my test? I'll try another run to see if that behavior is reproduced:

Thumbnails: https://web.lcrc.anl.gov/public/e3sm/diagnostic_output/ac.dcomeau/20220706.CRYO1850.ne30pg2_SOwISC12to60E2r4.chrysalis.CryoBranchN2DependentDefaultMinMaxKappa/yrs91-100/ocean/index.html#global_ohc_anom

PNG:
https://web.lcrc.anl.gov/public/e3sm/diagnostic_output/ac.dcomeau/20220706.CRYO1850.ne30pg2_SOwISC12to60E2r4.chrysalis.CryoBranchN2DependentDefaultMinMaxKappa/yrs91-100/ocean/index.html#global_ohc_anom&gid=6&pid=1

@xylar
Copy link
Collaborator Author

xylar commented Aug 16, 2022

@darincomeau, the purge may not be deleting the html directory. Could you try deleting it manually?

@xylar
Copy link
Collaborator Author

xylar commented Aug 16, 2022

Actually, the thumbnail looks right to me. I think if you do shift-F5 or the Mac equivalent in your browser, you'll find that it's just your browser caching the old thumbnail and if you force it to update it will be correct.

@darincomeau
Copy link
Contributor

Ah yes, that did it - thanks!

Copy link
Contributor

@darincomeau darincomeau left a comment

Choose a reason for hiding this comment

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

Approved based on above testing.

@xylar
Copy link
Collaborator Author

xylar commented Aug 16, 2022

Thanks @darincomeau!

@xylar xylar merged commit 2669d9d into MPAS-Dev:develop Aug 16, 2022
@xylar xylar deleted the fix_ohc_map branch August 16, 2022 19:58
@alicebarthel
Copy link

Great! Thanks @xylar and @darincomeau , that's very helpful!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OHC anomaly maps are blank
3 participants