8000 doc by high-in-entropy · Pull Request #2 · high-in-entropy/matplotlib · GitHub
[go: up one dir, main page]
Skip to content

doc #2

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Feb 28, 2019
Merged

doc #2

merged 4 commits into from
Feb 28, 2019

Conversation

high-in-entropy
Copy link
Owner
@high-in-entropy high-in-entropy commented Feb 28, 2019 •
edited
Loading

PR Summary

PR Checklist

  • Has Pytest style unit tests
  • Code is Flake 8 compliant
  • New features are documented, with examples if plot related
  • Documentation is sphinx and numpydoc compliant
  • Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there)
  • Documented in doc/api/api_changes.rst if API changed in a backward-incompatible way

anntzer and others added 4 commits February 27, 2019 18:44
@anntzer
More followup to autoregistering 3d axes.
73cb465 
I split out these as a separate PR for simpler review.
@anntzer
Simplify units.Registry.get_converter.
25cae8e 
The caching of the converter was disabled 10 years ago
(be73ef4) and is unlikely to ever work
as the correct converter for ndarrays depends on the type of the
contents, not on the container, so just drop the cache-related code.

When a masked array is passed, it's OK to just get the first underlying
data, even if it is masked (as it should have the same type as the
unmasked entries), for the purpose of getting the converter.
@timhoffm
Merge pull request #9314 from anntzer/simpler-units
cd3ed7a 
Simplify units.Registry.get_converter.
@timhoffm
Merge pull request #13534 from anntzer/more-mplot3d
2208665 
More followup to autoregistering 3d axes.
@high-in-entropy high-in-entropy merged commit 9a84637 into high-in-entropy:master Feb 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@high-in-entropy @anntzer @timhoffm
0