Skip to content

Backport PR #17422: Unstale viewlims before draw()ing polar axes. #17435

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

Merged

Conversation

tacaswell
Copy link
Member

Merge pull request #17422 from anntzer/polarunstale

Unstale viewlims before draw()ing polar axes.

Conflicts:
lib/matplotlib/projections/polar.py
- conflicts from nearyby changes to signature of draw
lib/matplotlib/tests/test_polar.py
- tests are still in test_axes.py on v3.2.x

… axes.

Merge pull request matplotlib#17422 from anntzer/polarunstale

Unstale viewlims before draw()ing polar axes.

Conflicts:
	lib/matplotlib/projections/polar.py
          - conflicts from nearyby changes to signature of draw
	lib/matplotlib/tests/test_polar.py
          - tests are still in test_axes.py on v3.2.x
@tacaswell tacaswell added this to the v3.2.2 milestone May 17, 2020
@tacaswell tacaswell added the Release critical For bugs that make the library unusable (segfaults, incorrect plots, etc) and major regressions. label May 17, 2020
@timhoffm
Copy link
Member

flake8 error seem unreated:
./examples/statistics/confidence_ellipse.py:220:21: F541 f-string is missing placeholders

@tacaswell
Copy link
Member Author

Pushed a fix, I think we can look forward to seeing that error on master in the very near future as well (the code is the same, it looks like flake8 got (correctly) stricter under us again.

@dstansby dstansby merged commit 391bb24 into matplotlib:v3.2.x May 18, 2020
@tacaswell tacaswell deleted the auto-backport-of-pr-17422-on-v3.2.x branch May 18, 2020 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release critical For bugs that make the library unusable (segfaults, incorrect plots, etc) and major regressions.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants