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

Normalized Taylor plot #79

Merged
merged 7 commits into from
Oct 1, 2021
Merged

Normalized Taylor plot #79

merged 7 commits into from
Oct 1, 2021

Conversation

jsmariegaard
Copy link
Member

@jsmariegaard jsmariegaard commented Sep 16, 2021

Sometimes it's more convenient to do a Taylor plot with a normalized std axis as it allows for comparing multiple stations in the same plot.

As a response to @dhithu's suggestion #77 ...

@jsmariegaard
Copy link
Member Author

Have now introduced "normalize" and "aggregate" keyword arguments to taylor plot:

image

@jsmariegaard
Copy link
Member Author

@dhithu What do you think? Was it something like this?

@dhithu
Copy link

dhithu commented Sep 20, 2021

@jsmariegaard Very nice, looking forward to trying this in a project! I have tested a bit and it seems to work nicely. I have two comments, the first is that I don't think the combination normalize=false and aggregate=false should be allowed, since it doesn't make sense. It is presently allowed, but the result doesn't look correct. The second comment is just a cosmetic one, it would look a bit clearer when normalize=true, if the std-axis tickmarks/grid included the value 1. Not sure if that's possible. See example here: https://www.frontiersin.org/files/Articles/389578/fmars-05-00287-HTML/image_m/fmars-05-00287-g005.jpg

@dhithu dhithu closed this Sep 20, 2021
@dhithu
Copy link

dhithu commented Sep 20, 2021

Here's my first normalized Taylor diagram :-)
output

@dhithu dhithu reopened this Sep 20, 2021
@jsmariegaard
Copy link
Member Author

@dhithu I have changed the name of the arguments to normalize_std and aggregate_observations to be more explicit

@jsmariegaard jsmariegaard marked this pull request as ready for review October 1, 2021 11:35
@jsmariegaard jsmariegaard merged commit a11a092 into main Oct 1, 2021
@jsmariegaard jsmariegaard deleted the normalized-taylor-plot branch October 1, 2021 11:40
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.

2 participants