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

Upgrade Taylor Diagram function #874

Merged
merged 17 commits into from
Oct 12, 2022
Merged

Upgrade Taylor Diagram function #874

merged 17 commits into from
Oct 12, 2022

Conversation

lee1043
Copy link
Contributor

@lee1043 lee1043 commented Sep 22, 2022

  • More flexibility (e.g., colormap)
  • New capability: Show arrow to compare pair of models

@lee1043 lee1043 linked an issue Oct 9, 2022 that may be closed by this pull request
@lee1043 lee1043 marked this pull request as ready for review October 12, 2022 20:08
@lee1043
Copy link
Contributor Author

lee1043 commented Oct 12, 2022

@acordonez could you please help on verifying this PR by rerunning update notebooks and see if they are reproducible from your end?

@acordonez
Copy link
Collaborator

@lee1043 Yes I can do this today.

Copy link
Collaborator

@acordonez acordonez left a comment

Choose a reason for hiding this comment

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

Looks nice! All ran smoothly on my end.

@lee1043
Copy link
Contributor Author

lee1043 commented Oct 12, 2022

@acordonez thanks!

@lee1043 lee1043 merged commit f92f92f into main Oct 12, 2022
@lee1043 lee1043 deleted the 871_ljw_taylor branch October 12, 2022 20:54
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.

Taylor diagram with arrows
2 participants