Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reduce unnecessary tests #7855

Closed
wants to merge 1 commit into from
Closed

Reduce unnecessary tests #7855

wants to merge 1 commit into from

Conversation

ebr
Copy link
Member

@ebr ebr commented Mar 28, 2025

Summary

  • Remove python 3.10 from the test matrix as its use will be deprecated, and to unblock Change timm and controlnet-aux pins to fix LLaVA model support #7846
  • Comment out tests that require GPU because they are meaningless on GPU-less runners. If we at some point launch GPU-backed test runners, these tests can be restored.
  • use ubuntu24.04 (latest LTS) in linux tests

NOTE: we need to remove these tests from required checks for the repo

Merge Plan

Merge when approved

Checklist

  • The PR has a short but descriptive title, suitable for a changelog

@github-actions github-actions bot added the CI-CD Continuous integration / Continuous delivery label Mar 28, 2025
@psychedelicious
Copy link
Collaborator

I believe this is superseded by #7739

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-CD Continuous integration / Continuous delivery
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants