Skip to content

Fix PIL.Image import in _check_cv_dep#377

Merged
keighrim merged 1 commit intodevelopfrom
376-fix-cv-import
Mar 11, 2026
Merged

Fix PIL.Image import in _check_cv_dep#377
keighrim merged 1 commit intodevelopfrom
376-fix-cv-import

Conversation

@keighrim
Copy link
Member

Fixing #376

Replace importlib.__import__ with importlib.import_module in
_check_cv_dep so that submodules (e.g. PIL.Image) are returned
directly instead of the top-level package.
@clams-bot clams-bot added this to infra Mar 11, 2026
@github-project-automation github-project-automation bot moved this to Todo in infra Mar 11, 2026
@codecov
Copy link

codecov bot commented Mar 11, 2026

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 73.24%. Comparing base (e0c85c3) to head (39ab95d).
⚠️ Report is 2 commits behind head on develop.

Files with missing lines Patch % Lines
mmif/utils/video_document_helper.py 50.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #377   +/-   ##
========================================
  Coverage    73.24%   73.24%           
========================================
  Files           22       22           
  Lines         3106     3106           
========================================
  Hits          2275     2275           
  Misses         831      831           
Flag Coverage Δ
unittests 73.24% <50.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@keighrim keighrim merged commit 0dc017f into develop Mar 11, 2026
7 of 8 checks passed
@github-project-automation github-project-automation bot moved this from Todo to Done in infra Mar 11, 2026
@keighrim keighrim mentioned this pull request Mar 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant