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
@keighrim keighrim deleted the 376-fix-cv-import branch March 13, 2026 01:08
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