Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
Propagate signature-modifying decorators to pyplot wrappers.#15254
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
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Uh oh!
There was an error while loading.Please reload this page.
b0f1695
to144b913
Compare144b913
to61a21ba
Compare61a21ba
tofbffe2f
Comparee8597f7
toce1eed0
CompareWhat is the disposition on this PR (asking in relation to#15049)? Looks like the conflict is in generated code, and can be resolved rather simply. |
There's no final decision between this and#14130 yet. In the end, I don't have a very strong opinion anymore on which one to choose. The important thing is that we get one of them in, so that we can properly rename parameters and propagate that to pyplot. Since this is internal functionality, we could even switch out the solution if we realize the other one was better. In practice, we need a second positive review so that this can get merged. The confict is indeed trivial to solve by regenerating the code. |
rebased |
I want to merge, but I'm wondering if there's an example that would trigger this yet? |
@QuLogic I think there are others, but one such example is the The argument (and docstring) were changed in the actual implementation, but pyplot still has the old kwarg (with the new docstring) |
Remove specification for where the old parameter remains supportedThis was fixed by mergingmatplotlib#15254
PR Summary
alternative to#14130, see discussion there; attn@timhoffm
PR Checklist