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

Tweaks to generating the examples gallery #583

Merged
merged 7 commits into from Aug 25, 2019
Merged

Tweaks to generating the examples gallery #583

merged 7 commits into from Aug 25, 2019

Conversation

newville
Copy link
Member

These are all changes to the doc and build scripts for the examples gallery.

I added "Gallery" to the upper-right section of the doc, just to make it more visible.

I tweaked the gallery generation script. In the end, this is mostly adding suppression of warnings.

I also changed the Makefile so that the gallery isn't generated every time sphinx-build is run, but only when the gallery is empty (as after make clean).

@reneeotten Let me know if any of these seem wrong or you don't like any part of this.

Type of Changes
  • Bug fix
  • New feature
  • Refactoring / maintenance
  • Documentation / examples
Tested on

Tested with all the sphinx tools up-to-date from Anaconda Python 3.7

Copy link
Contributor

@reneeotten reneeotten left a comment

Choose a reason for hiding this comment

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

thanks @newville! I like the idea of not generating the gallery every time even if there are no changes, so I like your Makefile-voodoo. I have left a few small review comments.

doc/sphinx/theme/lmfitdoc/layout.html Outdated Show resolved Hide resolved
doc/doc_examples_to_gallery.py Show resolved Hide resolved
doc/Makefile Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Aug 23, 2019

Codecov Report

Merging #583 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #583   +/-   ##
======================================
  Coverage    86.5%   86.5%           
======================================
  Files          11      11           
  Lines        3252    3252           
======================================
  Hits         2813    2813           
  Misses        439     439

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 10afd00...76f9575. Read the comment docs.

@newville newville merged commit 7b664fc into master Aug 25, 2019
@newville newville deleted the gallery_tweaks branch September 26, 2019 22:02
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.

None yet

2 participants