Skip to content

Conversation

@dopplershift
Copy link
Member

Description Of Changes

Our default was unintentionally overriding the matplotlib default, which is to (when contours are monochrome) to use solid lines for positive values and dashed lines for negative values. Change the default to None and better document the options.

Checklist

@dopplershift dopplershift added Type: Bug Something is not working like it should Area: Plots Pertains to producing plots labels Oct 20, 2022
@dopplershift dopplershift added this to the October 2022 milestone Oct 20, 2022
@dopplershift dopplershift requested review from a team and kgoebber as code owners October 20, 2022 17:50
@dopplershift dopplershift requested review from dcamron and removed request for a team October 20, 2022 17:50
Our default was unintentionally overriding the matplotlib default, which
is to (when contours are monochrome) to use solid lines for positive
values and dashed lines for negative values. Change the default to
`None` and better document the options.
Copy link
Collaborator

@kgoebber kgoebber left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks for updating this one!

Copy link
Member

@dcamron dcamron left a comment

Choose a reason for hiding this comment

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

👍

@dcamron dcamron merged commit 1e0542e into Unidata:main Oct 21, 2022
@dopplershift dopplershift deleted the declarative-dashed branch October 22, 2022 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Plots Pertains to producing plots Type: Bug Something is not working like it should

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement a shortcut for negative, dashed contours

3 participants