Skip to content

Napari points sam#48

Merged
k034b363 merged 10 commits intomainfrom
napari-points-sam
Feb 24, 2026
Merged

Napari points sam#48
k034b363 merged 10 commits intomainfrom
napari-points-sam

Conversation

@maliagehan
Copy link
Member

@maliagehan maliagehan commented Jan 30, 2026

Describe your changes
added a method to napari_read that formats data for input into ultralytics SAM functions.

Type of update
Is this a:

  • New feature or feature enhancement
  • Update to documentation

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

added function and tests, still need docs.
also fixed a problem with a link in naive bayes
fixed whitespace
@deepsource-io
Copy link

deepsource-io bot commented Jan 30, 2026

DeepSource Code Review

We reviewed changes in f9aa114...d302091 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  

Code Review Summary

Analyzer Status Updated (UTC) Details
Python Feb 24, 2026 9:08p.m. Review ↗
Test coverage Feb 24, 2026 9:11p.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 ↗

deepsource complained about using range instead of enumerate
@maliagehan maliagehan added enhancement Enhancements to existing features ready to review Ready for code review labels Jan 30, 2026
@k034b363 k034b363 merged commit 427710c into main Feb 24, 2026
5 checks passed
@k034b363 k034b363 deleted the napari-points-sam branch February 24, 2026 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Enhancements to existing features ready to review Ready for code review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants