Skip to content

Cap opencv version#1877

Merged
nfahlgren merged 1 commit intomainfrom
stopgap-opencv-version
Feb 16, 2026
Merged

Cap opencv version#1877
nfahlgren merged 1 commit intomainfrom
stopgap-opencv-version

Conversation

@joshqsumner
Copy link
Contributor

Describe your changes
New installations of PlantCV will get opencv 4.13 which breaks automatic color card detection. We have a fix in v5.0 but until that is ready for release it seems simplest to cap the opencv version at 4.12.0.

Type of update
This is a stopgap solution to a potential problem until v5 is released.

Associated issues
None, but see tripsicum seed head tutorial for error from opencv 4.13.0

Additional context
Had a fix in 1871 pointed at v5 but may need to revisit.

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

@deepsource-io
Copy link

deepsource-io bot commented Feb 16, 2026

DeepSource Code Review

DeepSource reviewed changes in the commit range 43ce1be..22c386d on this pull request. Below is the summary for the review, and you can see the individual issues we found as review comments.

For detailed review results, please see the PR on DeepSource ↗

PR Report Card

Security × 0 issues Overall PR Quality   

Reliability × 0 issues
Complexity × 0 issues
Hygiene × 0 issues
Coverage × 0 issues

Code Review Summary

Analyzer Status Summary Details
Python No new issues detected. Review ↗
Test coverage No new issues detected. Review ↗
How are these analyzer statuses calculated?

Administrators can configure which issue categories are reported and cause analysis to be marked as failed when detected. This helps prevent bad and insecure code from being introduced in the codebase. If you're an administrator, you can modify this in the repository's settings.

Code Coverage Summary

Language Line Coverage Branch Coverage
Aggregate
100%
100%
Python
100%
100%

➟ Additional coverage metrics may have been reported. See full coverage report ↗

@joshqsumner joshqsumner changed the title Update environment.yml Cap opencv version Feb 16, 2026
@joshqsumner joshqsumner added good first issue/PR Suitable for a first time contributor/reviewer ready to review labels Feb 16, 2026
@github-project-automation github-project-automation bot moved this to Pull Requests in PlantCV4 Feb 16, 2026
@nfahlgren nfahlgren added this to the PlantCV v4.10.3 milestone Feb 16, 2026
@nfahlgren nfahlgren merged commit ebe54de into main Feb 16, 2026
5 checks passed
@github-project-automation github-project-automation bot moved this from Pull Requests to Done in PlantCV4 Feb 16, 2026
@nfahlgren nfahlgren deleted the stopgap-opencv-version branch February 16, 2026 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

good first issue/PR Suitable for a first time contributor/reviewer ready to review

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants