Skip to content

Add visualize.height_distribution - #165

Open
k034b363 wants to merge 8 commits into
mainfrom
147-visualize-height
Open

Add visualize.height_distribution#165
k034b363 wants to merge 8 commits into
mainfrom
147-visualize-height

Conversation

@k034b363

Copy link
Copy Markdown
Contributor

Describe your changes
Our percentile-based plot height analysis function requires choosing an upper and lower percentile threshold to subtract for a plot-level height estimate. It would be helpful to be able to see some distributions of pixel values in individual plots in the DEM to help choose sensible cutoffs. This PR adds a sub-module called visualize and a function height_distribution that plots histograms for a random subset of plots in a provided shapefile.

Type of update

  • New feature or feature enhancement

Associated issues
Closes #147

For the reviewer
See this page for instructions on how to review the pull request.

  • PR functionality reviewed in a Jupyter Notebook
  • All tests pass
  • Test coverage remains 100%
  • Documentation tested
  • New documentation pages added to plantcv-geospatial/mkdocs.yml
  • Changes to function input/output signatures added to changelog.md
  • Code reviewed
  • PR approved

@k034b363 k034b363 added enhancement New feature or request Work in Progress labels Jul 28, 2026
@k034b363 k034b363 linked an issue Jul 28, 2026 that may be closed by this pull request
@deepsource-io

deepsource-io Bot commented Jul 28, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in bcc216d...a50be80 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Coverage  

Code Review Summary

Analyzer Status Updated (UTC) Details
Python Aug 13, 2026 8:07p.m. Review ↗
Code coverage Aug 13, 2026 8:08p.m. Review ↗

Code Coverage Summary

Language Line Coverage (New Code) Line Coverage (Overall)
Aggregate
100%
100%
Python
100%
100%

➟ Additional coverage metrics may have been reported. See full coverage report ↗


Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request ready to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add visualization to parameterize height percentile

1 participant