Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
issue: 26871 - Remove SimplePath class from patches.py#26876
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.
| @@ -0,0 +1,3 @@ | |||
| Remove Deprecated API | |||
| ~~~~~~~~~~~~~~~~~~~~~~~ | |||
| ``matplotlib.patches.SimplePath`` | |||
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.
Can you check the deprecation note an copy over the alternative if any?
Can you please squash these commits? https://matplotlib.org/devdocs/devel/development_workflow.html#rewrite-commit-history |
…catedWe don't need the optional packages and save a little download/installtime.
…catedtimedeltas make sense
5c6a2d5 to23f641cCompareThe git history here seems to be messy and includes unrelated commits from e.g.#26505, but overwrites their commit messages. We will also need to sort out between this and#26890, which also is doing the same deprecation task (and while that author closed/reopened several PRs, did get the first one as#26875, if we use that as the metric) |
Thank you for your efforts here, however we elected to merge another PR that took on this portion of this task. We hope to hear from you again |
PR summary
This PR removes the deprecated code during the version 3.7 from lib/matplotlib/patches.py which is a sub issue in the parent issues here:#26865
PR checklist