Skip to content

No longer test supergradients#649

Merged
guarin merged 13 commits intomainfrom
no-longer-test-supergradients
Mar 23, 2026
Merged

No longer test supergradients#649
guarin merged 13 commits intomainfrom
no-longer-test-supergradients

Conversation

@guarin
Copy link
Copy Markdown
Contributor

@guarin guarin commented Mar 18, 2026

What has changed and why?

  • Remove SuperGradients from CI tests as it is outdated and oftentimes causes issues
  • Excluded more albumentations versions
  • Skip running fine-tuning tests if albumentations version is too old

All excluded albumentations versions fail with:

ImportError: cannot import name 'preserve_channel_dim' from 'albucore.utils'

How has it been tested?

  • CI

Did you update CHANGELOG.md?

  • Yes
  • Not needed (internal change)

Did you update the documentation?

  • Yes
  • Not needed (internal change without effects for user)

@guarin guarin marked this pull request as ready for review March 23, 2026 12:50
Copilot AI review requested due to automatic review settings March 23, 2026 12:50
@guarin
Copy link
Copy Markdown
Contributor Author

guarin commented Mar 23, 2026

/review

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates CI/dependency setup to avoid unstable/outdated SuperGradients-related installs and to prevent fine-tuning test failures caused by incompatible albumentations versions (especially on Python 3.8).

Changes:

  • Skip fine-tuning integration tests when albumentations is below the supported version threshold.
  • Split albumentations dependency constraints by Python version and exclude 1.4.* where problematic.
  • Remove super-gradients from the default extras sets used by Makefile-based CI installs.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
tests/_commands/test_train_task.py Adds a module-level skip to avoid running training tests with too-old albumentations.
pyproject.toml Adjusts albumentations constraints by Python version and pins albumentations<1.4.0 for the super-gradients extra.
Makefile Removes super-gradients from CI/dev extras and updates related comments/docs in the dependency install targets.

Comment thread tests/_commands/test_train_task.py Outdated
guarin and others added 2 commits March 23, 2026 14:14
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@guarin guarin enabled auto-merge (squash) March 23, 2026 14:21
@guarin guarin merged commit 60c5e47 into main Mar 23, 2026
16 checks passed
@guarin guarin deleted the no-longer-test-supergradients branch March 23, 2026 14:33
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.

3 participants