-
Notifications
You must be signed in to change notification settings - Fork 288
Add pixi version to system requirements #1945
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
Comments
Do you mean like a pinned version, or a minimum required version? :) |
I think a pinned version, but maybe both? not sure the best way to handle it. I can give a concrete example Nerfstudio has the following pixi.toml https://github.com/nerfstudio-project/nerfstudio/blob/main/pixi.toml using 0.28.2 this fails for me and I get pixi install
⠠ default:linux-64 [00:00:01] resolving tomli==2.0.1 × failed to solve the pypi requirements of 'default' 'linux-64'
├─▶ failed to resolve pypi dependencies
├─▶ Failed to download and build `fire==0.6.0`
├─▶ Failed to install requirements from build-system.requires (install)
├─▶ Failed to prepare distributions
├─▶ Failed to fetch wheel: setuptools==69.5.1
├─▶ Failed to extract archive
╰─▶ Invalid gzip header
but with 0.25.0 it resolves no problem |
I like this idea! Note that the system requirements are mostly lower bound only. [system-requirements]
pixi = "<=0.25.0"
cuda = "<=12.1" # Errors But that might not be a huge problem. As it is non standard anyway. |
Yes, this exactly would be perfect! |
Now that pixi.toml recently broke forward compatibility, this would be particularly useful - we have users coming in with latest pixi and then not being able to use it because we're still on pixi 0.39 but 0.40 can't handle the file anymore. See |
Problem description
Often times I'll have a project that I made some time ago, which I think is still reproducible with its lock file, but fails because I'm using a different version of pixi. A potential solution may be to add an optional pixi version to the
system-requirements
table. This would help ensure folks don't run into situations where they think things will work further down the line, but because of a regression they dontThe text was updated successfully, but these errors were encountered: