-
Notifications
You must be signed in to change notification settings - Fork 10
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
Consistent bin-width in histograms summarising multiple images #524
Conversation
Closes #479 Contrary to the title `plotting.py`, which uses Seaborn, produces consistent bin-width when the range of values differs between images. The Jupyter Notebook (`notebooks/02-Summary-statistics-and-plots.ipynb`) uses Pandas for loading data and plotting it (a deliberate choice to reduce the number of packages users would encounter) and it is Pandas which produces the different bin-widths (see [#22222 ENH: groupby.hist bins don't match](pandas-dev/pandas#22222)). The Notebook has been updated to show how to use `np.linspace()` across the total range of data. Docstrings of `tests/test_plotting.py` have also been improved.
Codecov ReportPatch coverage:
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more Additional details and impacted files@@ Coverage Diff @@
## main #524 +/- ##
==========================================
+ Coverage 80.02% 80.08% +0.06%
==========================================
Files 19 19
Lines 2783 2792 +9
==========================================
+ Hits 2227 2236 +9
Misses 556 556
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ 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.
A useful clarification 😄
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.
This looks great and should clear things up! I am just wondering whether the commented out lines 296 and 297 are meant to be cleaned up?
Good catches on the commented code and grammar @Jean-Du thanks. Thanks for the approval @SylviaWhittle Merged 🎉 |
Closes #479
Contrary to the title
plotting.py
, which uses Seaborn, produces consistent bin-width when the range of values differs between images. The Jupyter Notebook (notebooks/02-Summary-statistics-and-plots.ipynb
) uses Pandas for loading data and plotting it (a deliberate choice to reduce the number of packages users would encounter) and it is Pandas which produces the different bin-widths (see #22222 ENH: groupby.hist bins don't match).The Notebook has been updated to show how to use
np.linspace()
across the total range of data.Docstrings of
tests/test_plotting.py
have also been improved.