-
Notifications
You must be signed in to change notification settings - Fork 416
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
Clean up High/Low Plotting Function A Bit #1234
Comments
I think this seems like some nice improvements. I'd be interested in seeing a PR go in to update the example. We could also look at incorporating the extrema-finding and the plotting as functions in MetPy. |
I figured it would be good place to have it since it seems to work and adds a nice graphic to surface and upper mb plots |
I would make a suggestion for the Hi/Low feature. The red colour does show well with the black highlight but it is hard to read the dark blue colour and black highlight. I would suggest a lighter blue colour with the black highlight to make easier to read. |
The High Low function introduced in the MSLP 1000-500mb Thickness plotting example is a wonderful touch to Python graphics. I thought maybe adding a black outline to the text would help it pop a bit more.
ex:
With
matplotlib.patheffects
andoutline_effect = [patheffects.withStroke(linewidth=2.5, foreground='black')]
I also added an
ax
argument to the code so it could be imported as an external library and plot to the current axes.Since adding an outline pushes the pressures too close to the symbol I gave it some space, there's probably a more accepted way but this worked for me in my case.
Thought I would pass my 2 cents on!
Sample Image:
Source code for my HiLo
The text was updated successfully, but these errors were encountered: