Skip to content

Conversation

@augustebaum
Copy link
Contributor

@augustebaum augustebaum commented Jan 13, 2025

I followed the sphinx-gallery docs.

I tried to place reset_mpl in a different module in the hopes of removing the warning about sphinx not being able to cache sphinx_gallery_conf, but I couldn't manage to make the import work. I noticed sklearn declares a function in conf.py as well so I don't think it's too big a deal.

Closes #675

Copy link
Contributor

@sylvaincom sylvaincom left a comment

Choose a reason for hiding this comment

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

Tested it, it works, thanks!

@sylvaincom sylvaincom merged commit ad34c2d into main Jan 13, 2025
5 of 9 checks passed
@sylvaincom sylvaincom deleted the 675-increase-mpl-figure-dpi branch January 13, 2025 15:39
rouk1 pushed a commit that referenced this pull request Jan 14, 2025
I followed the [sphinx-gallery
docs](https://sphinx-gallery.github.io/stable/advanced.html#resetting-before-each-example).

I tried to place `reset_mpl` in a different module in the hopes of
removing the warning about sphinx not being able to cache
`sphinx_gallery_conf`, but I couldn't manage to make the import work. I
noticed sklearn declares a function in `conf.py` as well so I don't
think it's too big a deal.

Closes #675
waridrox pushed a commit to waridrox/skore that referenced this pull request Apr 15, 2025
I followed the [sphinx-gallery
docs](https://sphinx-gallery.github.io/stable/advanced.html#resetting-before-each-example).

I tried to place `reset_mpl` in a different module in the hopes of
removing the warning about sphinx not being able to cache
`sphinx_gallery_conf`, but I couldn't manage to make the import work. I
noticed sklearn declares a function in `conf.py` as well so I don't
think it's too big a deal.

Closes probabl-ai#675
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(docs): Augment dpi of matplotlib in conf.py

4 participants