Skip to content

[v2] Remove SpeedTask #2425

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

Closed
Samoed opened this issue Mar 24, 2025 · 5 comments
Closed

[v2] Remove SpeedTask #2425

Samoed opened this issue Mar 24, 2025 · 5 comments
Labels
v2 Issues and PRs related to `v2` branch

Comments

@Samoed
Copy link
Member

Samoed commented Mar 24, 2025

Setuptools issue pypa/setuptools#4910
GPUtils anderskm/gputil#59 anderskm/gputil#60
Removed speed from install dependencies #2423

UPD: setuptools rollback this change

Soon we won't be able to install it propely. Maybe we need to remove it, create alternative

All packages with deprecated configurations are advised to move before 2026.

@Samoed
Copy link
Member Author

Samoed commented Mar 24, 2025

Also issue with pylate lightonai/pylate#103

@Samoed Samoed changed the title GPUtils not installable with setuptools>78 GPUtils using outdated format for setup Mar 24, 2025
@KennethEnevoldsen
Copy link
Contributor

Hmm, I was actually considering removing the speed tests from v2 (no one uses them, and they are distinctly different from other tasks). @isaac-chung added it originally so would love his opinion.

Removed speed from install dependencies #2423

Make sure that there is a warning trying to run the speed tasks.

@isaac-chung
Copy link
Collaborator

The original desires for a speed task were from #848. If we think these no longer apply, we should remove the related tasks and files.

@KennethEnevoldsen
Copy link
Contributor

I guess we figured out that size was probably a better proxy than runtime (given that people run it on different systems

@KennethEnevoldsen
Copy link
Contributor

So remove Speedtask and GPUtils for v2

@KennethEnevoldsen KennethEnevoldsen added the v2 Issues and PRs related to `v2` branch label Mar 27, 2025
@Samoed Samoed changed the title GPUtils using outdated format for setup [v2] Remove SpeedTask Mar 27, 2025
@Samoed Samoed closed this as completed Apr 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v2 Issues and PRs related to `v2` branch
Projects
None yet
Development

No branches or pull requests

3 participants