-
Notifications
You must be signed in to change notification settings - Fork 152
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
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 at 00_getting_started
and left a few comments.
2a29429
to
b3c1df5
Compare
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!
b3c1df5
to
b928a91
Compare
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.