Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Silence deprecation warning when importing finance in pylab#7524

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
tacaswell merged 1 commit intomatplotlib:v2.xfromefiring:pylab_finance_warning
Nov 29, 2016

Conversation

efiring
Copy link
Member

The discussion is in#7295.

There is still a problem, though: the deprecation message says to switch to the toolkit. How is the user supposed to know how to do that? It's not in the mpl_toolkits part of the distribution.

@efiringefiring added this to the2.0 (style change major release) milestoneNov 28, 2016
@NelleV
Copy link
Member

I don't really understand the problem: there are many toolkits that are not part of the distribution (cmocean for example).

@efiring
Copy link
MemberAuthor

efiring commentedNov 28, 2016
edited
Loading

Yes, but they are not toolkits that are beingremoved from mpl. The deprecation gives no clue as to where the new toolkit is, or even what it is called. Is it available anywhere other than via github?pip search cmocean shows that it exists under that name, and can be installed;pip search finance yields a long list, but I don't see anything that might be our finance toolkit.
Don't you think a user of finance.py might be puzzled upon seeing that deprecation message?

@NelleV
Copy link
Member

I'd indeed add the mpl_toolkits finance to pypi if it hasn't been done, to do a proper release. I can take care of this.
We've done something similar in slkearn, when we removed the hmm from sklearn and released it as a separate unmaintained project. There was not much confusion, but we did do a proper release and probably added a note somewhere to explain how to pip install it.

@NelleVNelleV changed the titleSilence deprecation warning when importing finance in pylab[MRG+1] Silence deprecation warning when importing finance in pylabNov 28, 2016
@tacaswelltacaswell merged commit5203df6 intomatplotlib:v2.xNov 29, 2016
@QuLogicQuLogic changed the title[MRG+1] Silence deprecation warning when importing finance in pylabSilence deprecation warning when importing finance in pylabNov 29, 2016
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@NelleVNelleVNelleV approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v2.0.0
Development

Successfully merging this pull request may close these issues.

3 participants
@efiring@NelleV@tacaswell

[8]ページ先頭

©2009-2025 Movatter.jp