Skip to content

Add option to plot time on x-axis in monitoring Hovmoeller plots #3732

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

Merged
merged 4 commits into from
Aug 6, 2024

Conversation

FranziskaWinterstein
Copy link
Contributor

@FranziskaWinterstein FranziskaWinterstein commented Aug 5, 2024

Description

This adds the option in the monitor script multi_datasets to create the hovmoeller plot hovmoeller_time_vs_lat_or_lon such that time is on the x-axis, which is more intuitive for time vs latitude. The default is time on y-axis and lat/lon on x-axis.
The new orientation can be chosen by the following setting in the recipe:

        plots:
          hovmoeller_time_vs_lat_or_lon:
            time_on: x-axis

hovmoeller_time_vs_lat_or_lon_EMAC_troz

Checklist

It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

New or updated recipe/diagnostic

Copy link
Contributor

@schlunma schlunma left a comment

Choose a reason for hiding this comment

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

Thanks @FranziskaWinterstein, this is almost ready. There is just a tiny issue with axis labels.

@FranziskaWinterstein
Copy link
Contributor Author

Yes, thank you for spotting this. (y)

@schlunma schlunma changed the title Add horizonal hovmoeller Add option to plot time on x-axis in monitoring Hovmoeller plots Aug 6, 2024
@schlunma schlunma added this to the v2.12.0 milestone Aug 6, 2024
Copy link
Contributor

@schlunma schlunma left a comment

Choose a reason for hiding this comment

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

I tested the new option with one dataset and also multiple datasets, works as expected. Thanks!!

@schlunma schlunma merged commit f8546fd into main Aug 6, 2024
8 checks passed
@schlunma schlunma deleted the add-horizonal-hovmoeller branch August 6, 2024 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants