-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
keyword arguments for Maptplotlib.pyplot.quiver #11793
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
Conversation
power-cycling to retry the test suite |
Can you please edit the PR title and text to explain what this pull request is about? While I appreciate that you referenced the issue, it would be really helpful to also have a short summary here. |
I'm unable to clear some image comparison tests, although the ones in tests/tests_collections.py::test_quiver_limits do. @story645 -Thank you. |
I think the much simpler solution is to do something like https://github.com/matplotlib/matplotlib/blob/v2.2.2/lib/matplotlib/axes/_axes.py#L2135, define
and call the matchers in order until one doesn't raise TypeError; the returned value gives the correct values of X, Y, U, V, C. |
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.
Proposing to reject per #11526 (comment). Thank you for your contribution, but I think the problem was ill-posed to start with... Sorry!
Closing per decision in #11526. |
PR Summary
Attempt at providing keyword arguments for Matplotlib.pyplot.quiver - Currently unable to pass image comparison tests #11526
PR Checklist