-
Notifications
You must be signed in to change notification settings - Fork 106
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
plot multiple glaciers in same plot #1508
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Possibly we could add a test for this in this test:
oggm/oggm/tests/test_workflow.py
Line 437 in c8bdf67
def test_plot_region_model(): |
Also: add an entry to whats new!
I added a test in |
Open a PR here: https://github.com/OGGM/oggm-sample-data/tree/master/baseline_images/freetype_28 Once merged, change the commit hash here: Line 72 in c8bdf67
|
It's actuall best to download the reference images from the test suite fails: they send a link |
In this PR I add the functionality to plot multiple glaciers in the same plot.
For this, a new grid is defined which contains all local gdir.grid completely. This new functionality can be used with the kwarg
extend_plot_limit
(e.g.plot_inversion(gdirs, extend_plot_limit=True)
). If this is used there will be no topographic shading!Closes #1007
(I have not added tests as it is only a plotting function)
whats-new.rst