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

matplotlib._cntr not found #418

Closed
MartinLuethi opened this Issue Mar 4, 2018 · 11 comments

Comments

Projects
None yet
2 participants
@MartinLuethi
Copy link

commented Mar 4, 2018

Hi

matplotlib does not contain a module called _cntr.
Please stick to standard modules that are easily installed.

This is matplotlib git version (pre 2.2)
version '2.2.0rc1+282.g3f3065d3a.dirty'

@fmaussion

This comment has been minimized.

Copy link
Member

commented Mar 5, 2018

Thanks for the report! This probably changed in the lastest Matpotlib version (2.2). Your installed package is still a release candidate and therefore has not been tested yet. I will try to implement a fix for this later today.

@fmaussion

This comment has been minimized.

Copy link
Member

commented Mar 5, 2018

OK, after doing some research: we should indeed find another way to compute the contours of our masks. I will dig into this, but the easiest for now would be to stick to mpl version 2.1

(mpl 2.2 is going to be officially released today or tomorrow, sigh...)

@MartinLuethi

This comment has been minimized.

Copy link
Author

commented Mar 5, 2018

@fmaussion

This comment has been minimized.

Copy link
Member

commented Mar 5, 2018

Thanks a lot for the heads up! It's always unfortunate to start with an error.

Thanks for considering OGGM for your class! If you are interested in idealized examples, a good way to start could be to have a look at the flowline example. This blog post is another example.

I think that one of the hardest bit is (as you've noticed) will be to get all the necessary python packages installed for all your students. I discussed a possible solution with Horst last time he came to visit, but if you are interested you can contact me offline or on the phone to discuss the deployment solution we have opted for our classes in Innsbruck.

@MartinLuethi

This comment has been minimized.

Copy link
Author

commented Mar 5, 2018

@MartinLuethi

This comment has been minimized.

Copy link
Author

commented Mar 5, 2018

@fmaussion

This comment has been minimized.

Copy link
Member

commented Mar 5, 2018

I haven't worked further yet, but maybe you could give me a hint how to do a flowline calculation on a known bedrock topography

I will work out an example for this (including known mass-balance). It should be quite easy to do

I consider using Jupyter notebooks for the class, running on a Jupyterhub instance. This worked nicely for other stuff.

Yes! If you have full control on the packages installed in the background (ideally with conda) then this would work fine.

@fmaussion

This comment has been minimized.

Copy link
Member

commented Mar 5, 2018

This should now be fixed (#421).

See also #420 for a discussion about making some dependencies optional.

@MartinLuethi

This comment has been minimized.

Copy link
Author

commented Mar 6, 2018

@fmaussion fmaussion referenced this issue Mar 6, 2018

Merged

Add example notebook for flowline model #427

2 of 2 tasks complete
@fmaussion

This comment has been minimized.

Copy link
Member

commented Mar 7, 2018

Martin, I just added another example to the notebooks folder (https://github.com/OGGM/oggm/blob/master/docs/notebooks/flowline_with_known_bedrock.ipynb)

Let me know if something is unclear, and good luck with your lectures!

@MartinLuethi

This comment has been minimized.

Copy link
Author

commented Mar 7, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.