-
Notifications
You must be signed in to change notification settings - Fork 129
Pin libgcc for windows installs #1399
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
base: main
Are you sure you want to change the base?
Conversation
575494c
to
9bfb689
Compare
@@ -51,6 +51,7 @@ dependencies = [ | |||
"scipy>=1,<2", | |||
"numpy>=1.17.0", | |||
"filelock>=3.15", | |||
"libgcc<15 ; platform_system == 'Windows'", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The c backend is optional for PyTensor, this is something you want to do at the conda level not here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ya, libgcc isn't even on PyPI. Would anyone be available for a quick call to explain to me what's going on?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep I quickly discovered this PR is extremely stupid :)
I can close it once we have another one addressing the actual issue.
We only do a smoke test on mac but why is it redundant. They are very different oses these days, so much mac breaks every other day and I don't remember the last time we had issues on linux |
I did a bit more digging and I have a vague hypothesis. I don't think it's a matter of pinning libgcc, but rather that pytensor is finding the wrong compiler. Perhaps we are picking up the system compiler, or there is a mismatch within the conda environment itself between the gxx version and libgcc version? (ping @lucianopaz) (Note that we had to change to g++ on Windows, but it has been apparently working well for several months.) conda-forge/pytensor-suite-feedstock#133 (comment) I reproduced a Windows environment via pixi, and I'm wondering why Environment list
|
I checked on linux-64 and it was the same, but it still works on Linux. 🤷 |
Closes #1398 . Issue was reported and debugged by user DrEntropy here.
Maybe there's a better solution for this issue, but it's probably good to patch it quickly for now. We should also consider running the CI on windows -- testing linux and mac is a bit redundant?
📚 Documentation preview 📚: https://pytensor--1399.org.readthedocs.build/en/1399/