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

How ._cntr should be imported on Matplotlib (LTS) 2.2.0 #10709

Closed
arnaldorusso opened this issue Mar 7, 2018 · 6 comments

Comments

Projects
None yet
5 participants
@arnaldorusso
Copy link

commented Mar 7, 2018

In order to construct GeoJson, a package uses

import matplotlib._cntr as cntr

While using matplotlib 2.2.0 it returns

ImportError: No module named _cntr

How it should be imported?
Using the last LTS of matplotlib 2.1.0, it works.

@tacaswell

This comment has been minimized.

Copy link
Member

commented Mar 7, 2018

This was a private module and should not have been used by third-party packages. See https://github.com/matplotlib/legacycontour for an effort to package that code independently.

@tacaswell tacaswell closed this Mar 7, 2018

@WeatherGod

This comment has been minimized.

Copy link
Member

commented Mar 7, 2018

@arnaldorusso

This comment has been minimized.

Copy link
Author

commented Mar 9, 2018

Thank's @tacaswell and @WeatherGod . I'll see how it properly works in this case.

@TD22057

This comment has been minimized.

Copy link
Contributor

commented Apr 20, 2018

Just an FYI: we used _cntr as well. We have an application (QT GUI) where the user has interactive control of the contour lines and we used the functions provided by that module (i.e. getting the path for arbitrary contour levels without drawing a plot). It appears that I can swap to _contour and use that (and yes - I know we shouldn't use private calls). I think that if the contourc code in issue #367 were added/merged, then that would provide a public API for doing this.

@WeatherGod

This comment has been minimized.

Copy link
Member

commented Apr 20, 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.