Skip to content

Parallel inf images#1918

Merged
nfahlgren merged 3 commits into
v5.0from
parallel-INF-images
May 11, 2026
Merged

Parallel inf images#1918
nfahlgren merged 3 commits into
v5.0from
parallel-INF-images

Conversation

@joshqsumner
Copy link
Copy Markdown
Contributor

@joshqsumner joshqsumner commented May 11, 2026

Describe your changes
Allows for INF files to be found by parallel by default when requested with uppercase. Previously they could be found if you specified the imgformat AND turned include_all_subdirs to False, which is a little buggy.

Type of update
This is a feature enhancement.

Associated issues
None

Additional context
Came up working with jupyterconfig on a photosynthesis workflow, which might be the only use case for this.

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 added this to the PlantCV v5.0 milestone May 11, 2026
@joshqsumner joshqsumner added the enhancement Enhancements to existing features label May 11, 2026
@joshqsumner joshqsumner changed the base branch from main to v5.0 May 11, 2026 16:29
@deepsource-io
Copy link
Copy Markdown

deepsource-io Bot commented May 11, 2026

DeepSource Code Review

We reviewed changes in e90ca5f...a73eedd 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 May 11, 2026 6:42p.m. Review ↗
Code coverage May 11, 2026 6:42p.m. Review ↗

Code Coverage Summary

Language Line Coverage (Overall)
Aggregate
99.9%
[▼ down 0.1% from main]
Python
99.9%
[▼ down 0.1% from main]

➟ 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.

@nfahlgren nfahlgren merged commit 182d40c into v5.0 May 11, 2026
5 checks passed
@nfahlgren nfahlgren deleted the parallel-INF-images branch May 11, 2026 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Enhancements to existing features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants