8000 EXA Use tick_labels in boxplot for matplotlib>=3.9 by nithish08 · Pull Request #29471 · scikit-learn/scikit-learn · GitHub
[go: up one dir, main page]
Skip to content

EXA Use tick_labels in boxplot for matplotlib>=3.9 #29471

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
Jul 15, 2024
Merged

EXA Use tick_labels in boxplot for matplotlib>=3.9 #29471

merged 4 commits into from
Jul 15, 2024

Conversation

nithish08
Copy link
Contributor
@nithish08 nithish08 commented Jul 11, 2024 •
edited by lesteve
Loading

Make the examples run without DeprecationWarning for matplotlib>=3.9, see #29434.

@github-actions GitHub Actions
Copy link
github-actions bot commented Jul 11, 2024 •
edited
Loading

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: 73b3ada. Link to the linter CI: here

@nithish08 nithish08 mentioned this pull request Jul 11, 2024
Closed
3 tasks
@lesteve lesteve changed the title CI handle labels for matplotlib>=3.9 EXA handle labels for matplotlib>=3.9 Jul 15, 2024
@lesteve lesteve changed the title EXA handle labels for matplotlib>=3.9 EXA Use tick_labels in boxplot for matplotlib>=3.9 Jul 15, 2024
@lesteve
Copy link
Member
lesteve commented Jul 15, 2024 •
edited
Loading

Thanks, I have pushed a tweak for the comment. I have enabled auto-merge, so this PR will be merged when CI is green.

Side-comment: you don't really need to merge main into your branch. The main reason you would need to do this only if there are conflicts, i.e. you changed some lines that have been changed in main since you created your branch. In other cases, a maintainer will ask you explicitly to merge main into your branch if there is a need.

@lesteve lesteve enabled auto-merge (squash) July 15, 2024 05:05
@lesteve lesteve merged commit 13a1791 into scikit-learn:main Jul 15, 2024
34 checks passed
@nithish08 nithish08 deleted the nb/unpin-matplotlib branch July 15, 2024 15:01
@StefanieSenger StefanieSenger mentioned this pull request Jul 23, 2024
Merged
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Sep 9, 2024
@nithish08 @lesteve @glemaitre
EXA Use tick_labels in boxplot for matplotlib>=3.9 (scikit-learn#29471)
d11dcde 
Co-authored-by: Loïc Estève <loic.esteve@ymail.com>
glemaitre pushed a commit that referenced this pull request Sep 11, 2024
@nithish08 @lesteve @glemaitre
EXA Use tick_labels in boxplot for matplotlib>=3.9 (#29471)
436cb76 
Co-authored-by: Loïc Estève <loic.esteve@ymail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lesteve lesteve lesteve approved these changes

Assignees
No one assigned
Labels
Build / CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nithish08 @lesteve
0