Bound the OpenCV major and refresh pinned dependency versions#460
Merged
Conversation
je_open_cv leaves opencv-python unpinned, so the OpenCV 5.0 release silently changed cv2 return shapes and broke line/text-region detection for every fresh install; an explicit `<6` bound stops the next major from doing the same. Fold in the dependency bumps dependabot proposed against stale bases (pillow 12.3.0, pyobjc 12.2.1, ruff 0.15.22, pytest 9.1.1) and keep the versions the workflows hardcode in sync with dev_requirements.txt. Pinning the CI tooling installs also clears the SonarCloud githubactions:S8541/S8544 findings.
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 0 |
| Duplication | 0 |
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
pytest 9.1 changed `importorskip`'s default `exc_type` from ImportError to ModuleNotFoundError, so the container — which has PySide6 installed but no libEGL — turned 16 previously skipped GUI modules into collection errors. Ask for the old behaviour explicitly on the Qt guards. Also move the workflow NOSONAR justifications onto the flagged lines; a comment on the preceding line does not suppress githubactions:S8544.
|
This was referenced Jul 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Consolidates the stale dependabot bumps and closes the gap that let OpenCV 5 break
main.opencv-python>=4.8,<6as a direct dependency.je_open_cv==0.0.22leaves it unpinned, so 5.0.0.93 landed on 2026-07-19 and changedHoughLinesPfrom(N, 1, 4)to(N, 4), breaking line/text-region detection on every fresh install. Verified locally:test_edge_lines_batch.py+test_text_regions_batch.pypass under cv2 5.0.0 with the current sources.dev_requirements.txt— they had drifted becausequality.ymlhardcodes its tool versions.--only-binary :all:, clearing SonarCloudgithubactions:S8541/S8544on the release PR.uv.lockregenerated withuv lock.