Skip to content

Use matplotlib oo-style in examples#7531

Merged
jarrodmillman merged 13 commits intoscikit-image:mainfrom
GParolini:oo-matplotlib-segmentation
Feb 27, 2025
Merged

Use matplotlib oo-style in examples#7531
jarrodmillman merged 13 commits intoscikit-image:mainfrom
GParolini:oo-matplotlib-segmentation

Conversation

@GParolini
Copy link
Copy Markdown
Contributor

@GParolini GParolini commented Sep 6, 2024

Description

Checklist

Release note

Edit gallery examples on segmentation to comply with matplotlib's object-oriented style.

@mkcor mkcor added the 📄 type: Documentation Updates, fixes and additions to documentation label Oct 28, 2024
@lagru lagru self-requested a review October 28, 2024 15:12
@GParolini GParolini marked this pull request as ready for review October 29, 2024 23:04
@lagru
Copy link
Copy Markdown
Member

lagru commented Dec 17, 2024

It looks like most of the failures after @mkcor's update are described by #7636. Not sure yet, what pre-commit.ci is complaining about. It doesn't say which files it reformats...

@mkcor
Copy link
Copy Markdown
Member

mkcor commented Jan 8, 2025

The linux-cp3.13-default check is failing just like #7630 (comment), so I would consider it unrelated to this PR.

@mkcor mkcor requested a review from lagru January 8, 2025 16:32
@mkcor
Copy link
Copy Markdown
Member

mkcor commented Feb 25, 2025

@GParolini all CI checks are passing! We're almost there! 🎉

Do you mind addressing the one minor suggestion I left? Thanks! 🙏

Co-authored-by: Marianne Corvellec <marianne.corvellec@ens-lyon.org>
Copy link
Copy Markdown
Contributor

@jarrodmillman jarrodmillman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!


def make_ax(grid=False):
ax = plt.figure().add_subplot(projection='3d')
fig, ax = plt.subplots(subplot_kw={'projection': '3d'})
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just checking if this is a newer syntax?

@stefanv
Copy link
Copy Markdown
Member

stefanv commented Feb 27, 2025

I think we can merge this. The 3d setup is not important, I was just curious.

Thanks @GParolini!

@jarrodmillman jarrodmillman changed the title Docs: segmentation examples edited to comply with matplotlib oo-style Use matplotlib oo-style in examples Feb 27, 2025
@jarrodmillman jarrodmillman merged commit c6ac186 into scikit-image:main Feb 27, 2025
22 checks passed
@stefanv stefanv added this to the 0.26 milestone Feb 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📄 type: Documentation Updates, fixes and additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants