- Notifications
You must be signed in to change notification settings - Fork180
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Refactor tutorials: remove outputs from tutorials.#1266
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@## main #1266 +/- ##==========================================- Coverage 86.05% 78.28% -7.78%========================================== Files 118 119 +1 Lines 8672 8698 +26 ==========================================- Hits 7463 6809 -654- Misses 1209 1889 +680
Flags with carried forward coverage won't be shown.Click here to find out more. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Thanks a ton, this is much better!! I only had a look at00_getting_started and left a few comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Good to go now!
small fixes to getting_started tutorial
I wrote a custom nb preprocessor that strips everything except plots and prints from all notebooks.
I will add this script to CI for building the docs in a separate PR.