Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a scraper for pyvista #91

Merged
merged 8 commits into from
Dec 21, 2023
Merged

Add a scraper for pyvista #91

merged 8 commits into from
Dec 21, 2023

Conversation

Louis-Pujol
Copy link
Contributor

Hi ! I create a new PR as I forked again the project after some troubles with conflicting versions of the code.

In this PR I propose an implementation of a scraper for the pyvista library.

The scraper is a slight modification of the pyvista's sphinx-gallery scraper. I integrated it following the mayavi scraper already implemented.

It allows to show static or animated 3d scenes created with pyvista. I added an example to the examples gallery. I moved the definition of image_scrapers from mkdocs.yml to gallery_conf.py. I found this clearer as pyvista is configured in gallery_conf.py.

I also added pyvista as a dependency in MKDOCS_GALLERY_EXAMPLES_REQS in noxfile.py.

@Louis-Pujol
Copy link
Contributor Author

Seems to work, should I add a line to the 0.8.0 section of the changelog file or create a section 0.8.1 ?

@smarie
Copy link
Owner

smarie commented Dec 20, 2023

Thanks @Louis-Pujol ! Yes, please create a new changelog section (0.9.0) with this item. This feature is significant enough to modify the minor version number and not the patch version number :)

@Louis-Pujol
Copy link
Contributor Author

Louis-Pujol commented Dec 20, 2023

Ok I do that. In the meantine, I had also work on the following :

Would you prefer me to create new PRs for these changes, or to merge them directly here ?

@smarie
Copy link
Owner

smarie commented Dec 21, 2023

Thanks ! No, please Create distinct PRs, that will be easier to review

docs/changelog.md Outdated Show resolved Hide resolved
@smarie smarie merged commit 310ea71 into smarie:main Dec 21, 2023
11 checks passed
@smarie
Copy link
Owner

smarie commented Dec 21, 2023

0.9.0 released with your contribution : https://smarie.github.io/mkdocs-gallery/generated/gallery/plot_11_pyvista/
Thanks !

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.

2 participants