-
-
Notifications
You must be signed in to change notification settings - Fork 313
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
add xformers to cirun #1011
add xformers to cirun #1011
Conversation
If you just need more time, shouldn't it suffice with |
It's a CUDA build, so I think it needs the GPU agents? Not sure if It's possible that things might also work with a |
CUDA builds typically can be done on CPU. This actually is how RAPIDS builds its packages Think That said, if there is a testing need, it could make sense to use the GPU runners for that On a different note, there are a few other feedstocks here that could be moved to CPU only. Noted them in a separate issue: #1012 |
Are we OK now with trying the CPU runners? Or should we add both in this PR (but prefer the CPU one on the feedstock if it works)? |
Oops, almost forgot about this. Merging! |
A new PR will pop up in https://github.com/conda-forge/.cirun/pulls. Feel free to merge that if I don't see it before :P |
Done. |
xformers compiles a lot of symbols per architecture, and times out with our default set of architectures, which should look like this:
To avoid timing out with the 6h limit on azure, we're currently just able to build
i.e. 2-4 instead of 9 GPU architectures. I think it would be good to build the full set of architectures on the cirun server. Shouldn't need a particularly beefy machine, just more time.
CC @jaimergp @isuruf @jakirkham @conda-forge/xformers