-
Notifications
You must be signed in to change notification settings - Fork 271
New function analyze.distribution #1473
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
Conversation
Added new function analyze.distribution. Added documentation for analyze.distribution.
Here's the code health analysis summary for commits Analysis Summary
Code Coverage Report
|
Updates to distribution.py to fix errors in numpy to python list for the histogram axes.
Closer but not quite there still with this code:
|
Fixed bugs to allow plotting, labels plots by X or Y distribution
…er/plantcv into detection-issue1460
…er/plantcv into detection-issue1460
The debug image of the bounding rectangle was not needed and caused an issue when used in analyze.distribution
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.
Tested with labeled masks from bean tutorial and with unlabeled masks from minirhizotron, both worked. Used as a test to collect mean depth data from a set of minirhizotron images and it's very fast and flexible. Clarified a few minor things in the doc and doc strings, but it looks good to me!
Added new function analyze.distribution. Added documentation for analyze.distribution.
Describe your changes
This new function provides a spatial analysis of pixel distribution of a mask in the X and Y dimension. This will be most useful for minirhizotron root images.
Type of update
Associated issues
Issue #1460
Additional context
For the reviewer
See this page for instructions on how to review the pull request.
plantcv/mkdocs.yml
updating.md