Skip to content

case insensitive extensions for imgformat - #1990

Merged
nfahlgren merged 1 commit into
mainfrom
lowercase-only-ext
Aug 5, 2026
Merged

case insensitive extensions for imgformat#1990
nfahlgren merged 1 commit into
mainfrom
lowercase-only-ext

Conversation

@joshqsumner

Copy link
Copy Markdown
Contributor

Describe your changes
Coerce all extensions to lowercase to match with ext.lower in pcv.parallel.parsers

Type of update
This is a bug fix

Associated issues
None

Additional context
None

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 the bug Bug or potential bug reports label Aug 4, 2026
@deepsource-io

deepsource-io Bot commented Aug 4, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in 3250746...09bee18 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 4, 2026 6:52p.m. Review ↗
Code coverage Aug 4, 2026 6:52p.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.

@github-project-automation github-project-automation Bot moved this to Pull Requests in PlantCV4 Aug 5, 2026
@nfahlgren nfahlgren added bugfix Bug fixes and removed bug Bug or potential bug reports labels Aug 5, 2026
@nfahlgren nfahlgren added this to the PlantCV v4.11.4 milestone Aug 5, 2026
@nfahlgren
nfahlgren merged commit dae7f5e into main Aug 5, 2026
5 checks passed
@nfahlgren
nfahlgren deleted the lowercase-only-ext branch August 5, 2026 21:33
@github-project-automation github-project-automation Bot moved this from Pull Requests to Done in PlantCV4 Aug 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Bug fixes

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants