Skip to content

Pc001 multispec prototype - #1997

Open
joshqsumner wants to merge 9 commits into
v5.0from
pc001-multispec
Open

Pc001 multispec prototype#1997
joshqsumner wants to merge 9 commits into
v5.0from
pc001-multispec

Conversation

@joshqsumner

@joshqsumner joshqsumner commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Describe your changes
adds MS_data class and pcv.multispec module containing pcv.multispec.read_ms to read files/directories/lists of files into MS_data objects.

Type of update
This is a new feature.

Associated issues
None

Additional context
This is a prototype of how we might internally handle new multispectral data from the Pheno-Core's new MS camera given the metadata layout we are using there.

An example dataset is downloaded on stargate at /data/datascience/users/jsumner/pc001_test/images/.

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 updating.md
  • Code reviewed
  • PR approved

@joshqsumner
joshqsumner changed the base branch from main to v5.0 August 17, 2026 18:36
@joshqsumner joshqsumner added the new feature New feature ideas and solutions label Aug 17, 2026
@joshqsumner joshqsumner added this to the PlantCV v5.0 milestone Aug 17, 2026
@deepsource-io

deepsource-io Bot commented Aug 17, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in ffb5de8...660cb8b 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 25, 2026 8:03p.m. Review ↗
Code coverage Aug 25, 2026 8:03p.m. Review ↗

Code Coverage Summary

Language Line Coverage (New Code) Line Coverage (Overall)
Aggregate
100%
[✓ above threshold]
100%
Python
100%
[✓ above threshold]
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

new feature New feature ideas and solutions ready to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant