-
Notifications
You must be signed in to change notification settings - Fork 440
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
Fix Plotter docs #3968
Fix Plotter docs #3968
Conversation
Codecov Report
@@ Coverage Diff @@
## main #3968 +/- ##
=======================================
Coverage 95.56% 95.56%
=======================================
Files 94 94
Lines 20214 20214
=======================================
Hits 19317 19317
Misses 897 897 |
Should we add a check for broken intersphinx links? These are hard to catch manually. |
I'd like to have this merged today for our 0.38.4 release. As this doesn't change the API and really improves the docs, a cursory review would be appreciated but I'll press without it and beg for forgiveness. |
Alright, except for a few exceptions, fixed all the intersphinx and role links. |
* bump to 0.38.4 * update req in earnest * fix plotter reopening (#4102) * Update testing-and-deployment.yml * Update requirements_test.txt Co-authored-by: MatthewFlamm <39341281+MatthewFlamm@users.noreply.github.com> * Update testing-and-deployment.yml * Apply suggestions from code review * Change source links to github (#4113) * add linkcode * Update pyvista/utilities/docs.py Co-authored-by: Tetsuo Koyama <tkoyama010@gmail.com> --------- Co-authored-by: Tetsuo Koyama <tkoyama010@gmail.com> * Fix Plotter docs (#3968) --------- Co-authored-by: MatthewFlamm <39341281+MatthewFlamm@users.noreply.github.com> Co-authored-by: Tetsuo Koyama <tkoyama010@gmail.com> Co-authored-by: Bane Sullivan <bane.sullivan@kitware.com>
An attempt to resolveResolves #3956 and #4023