-
Notifications
You must be signed in to change notification settings - Fork 1
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
Bumped scipp to 23.08 #65
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #65 +/- ##
========================================
Coverage 97.86% 97.86%
========================================
Files 41 41
Lines 3050 3050
========================================
Hits 2985 2985
Misses 65 65 ☔ View full report in Codecov by Sentry. |
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.
Looks good. Two minor issues raised, once resolved the branch can be merged.
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.
looks good!
New version of scipp uses plopp as default plotting tool.
plot.py
In pyproject.toml:
Adjusted jupyter notebooks to new plotting functionality.
Adjust read in data to the more strict requirements by scipp (removed plural s in example data)