Skip to content

fix: add missing albumentations dependency to fix Codecov CI#97

Open
geetanunakani wants to merge 1 commit into
Brijeshthummar02:masterfrom
geetanunakani:fix/codecov-ci-albumentations
Open

fix: add missing albumentations dependency to fix Codecov CI#97
geetanunakani wants to merge 1 commit into
Brijeshthummar02:masterfrom
geetanunakani:fix/codecov-ci-albumentations

Conversation

@geetanunakani

Copy link
Copy Markdown
Contributor

Overview

Add missing albumentations dependency to fix Codecov CI failure.

Description

  • requirements.txt — added albumentations>=1.3.0

Core Motivation

utilities.py imports albumentations but it was absent from
requirements.txt, so the CI environment never installed it.
This caused ModuleNotFoundError at test collection time, making
both test_utilities_metrics.py and test_utilities_tta.py fail
before a single test could run — breaking Codecov CI on all PRs.

Proposed Changes

  1. Identified root cause in CI logs: ModuleNotFoundError: No module named 'albumentations'
  2. Confirmed albumentations is imported in utilities.py but missing from requirements.txt
  3. Added albumentations>=1.3.0 to requirements.txt so CI installs it before running pytest

Related Issue

Closes #96

Type of Change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

  • Checked changes locally
  • Verified root cause directly in the GitHub Actions CI log

Screenshots (if applicable)

N/A

Checklist

  • My code follows the code style of this project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • My changes generate no new warnings.

utilities.py imports albumentations but it was missing from
requirements.txt, causing ModuleNotFoundError during CI test
collection. Both test_utilities_metrics.py and test_utilities_tta.py
failed at import time due to this missing dependency.

Fix: add albumentations>=1.3.0 to requirements.txt so the CI
environment installs it before running pytest.

Fixes Brijeshthummar02#96
@Brijeshthummar02

Copy link
Copy Markdown
Owner

no code diff , check it

@Brijeshthummar02

Copy link
Copy Markdown
Owner

@geetanunakani are u fixing or should i close the PR?

@geetanunakani

Copy link
Copy Markdown
Contributor Author

Hi @Brijeshthummar02! Apologies for the delay — this is ready to go, no further changes needed from my side. The fix adds the missing albumentations dependency to requirements.txt to resolve the Codecov CI failure (closes #96). Please feel free to merge whenever convenient. Thanks for checking in! 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] - Codecov ci fails

2 participants