Skip to content

Comments

Napari bounding box#47

Open
k034b363 wants to merge 5 commits intomainfrom
napari-bounding-box
Open

Napari bounding box#47
k034b363 wants to merge 5 commits intomainfrom
napari-bounding-box

Conversation

@k034b363
Copy link
Contributor

Describe your changes
Adds a function that takes a napari viewer object with a shapes layer that has rectangular polygons and outputs a list of x, y, height, and width of the rectangle suitable for either defining an ROI or specifying a bounding box for segment anything.

Type of update

  • New feature or feature enhancement

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/mkdocs.yml
  • Changes to function input/output signatures added to changelog.md
  • Code reviewed
  • PR approved

@deepsource-io
Copy link

deepsource-io bot commented Jan 20, 2026

Here's the code health analysis summary for commits afd540f..2175cbd. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Python LogoPython✅ SuccessView Check ↗
DeepSource Test coverage LogoTest coverage✅ SuccessView Check ↗

Code Coverage Report

MetricAggregatePython
Branch Coverage100%100%
Composite Coverage100%100%
Line Coverage100%100%
New Branch Coverage100%100%
New Composite Coverage100%100%
New Line Coverage100%100%

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@k034b363 k034b363 added new feature New feature ideas and solutions ready to review Ready for code review labels Jan 20, 2026
@k034b363 k034b363 requested a review from maliagehan January 20, 2026 22:08
@maliagehan maliagehan self-assigned this Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new feature New feature ideas and solutions ready to review Ready for code review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants