ENH: update package version to 13.3.1#196
Conversation
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
….29 and conda-forge-pinning 2026.05.29.18.35.1
|
@acosmicflamingo quick question: the bot updated @leofang for visibility |
|
Ah right that should've gone to 0 when the version was bumped; my apologies! |
No worries, innovation is always an adventure :-) I can easily reset that to zero before merging this PR. You called this out before I think, this job is taking a very long time: https://github.com/conda-forge/cuda-python-feedstock/actions/runs/26668235358/job/78606076893?pr=196 IIUC you called this out already as expected. Is there a possibility that we can split it up in the future? |
Reset the build number for the new upstream release and align the v1 recipe with the published cuda-python metadata by adding cuda-core as a runtime dependency of the metapackage. Address follow-up review from the rattler-build migration by adding aarch64-only cudla runtime/test coverage, keeping libnvfatbin as a runtime dependency only, and restoring the setuptools workaround comment.
|
@rwgk hm that is interesting...with a conda-build run, even if each process builds its own noarch cuda-python package, they all have the same hash (which isn't surprising). I wonder if rattler-build and conda smithy think it'd be more efficient to have one run build cuda-python once and build Python variants of cuda-bindings (which is why it's taking so long), instead of have them all build in parallel, even if it means you are unnecessarily building the noarch package more than once. That's something I could look into when I'm by a computer sometime this weekend. |
It might be possible and if this is true it'd explain my confusion (#183 (comment)). I wouldn't sweat in this PR, though. It can be addressed at a later time. |
|
I'll create a ticket and investigate when I have a chance. What a huge bummer that there's huge performance gains in the builds themselves, just for rattler-build to be too clever for its own good here 🥲 |
|
I already asked my bot to start analyzing it through the experiment in #198 |
|
Hi! This is the friendly conda-forge automerge bot! I considered the following status checks when analyzing this PR:
Thus the PR was passing and merged! Have a great day! |
I think I fix the parallelization but still am waiting to check if the same hash for the noarch packages would be generated. Feel free to merge this PR as-is. |
|
Wait...no, it failed. Actually, that error was what led me to realize that the noarch hashes differed when I was explicitly adding is_freethreading in the recipe.yaml. But, maybe you're in the right direction 🧐 |
|
Looked into this — Both options write to the same // Add use_keys to the variant (forces them to be included even if not referenced)
...
for key in &use_keys {
if let Some(value) = combination.get(key) {
variant.insert(key.clone(), value.clone());
}
}
// Filter out ignore_keys from the variant
...
variant.retain(|key, _| !ignore_keys.contains(key));Net result for There's no two-stage semantics (e.g. "use for matrix, ignore for hash") — both options write to the single So the only ways to get "matrix splits on python, but hash doesn't include python" today are:
-- Leo's bot |
|
Confirmed that #198 fixed/restored the parallel builds! |

Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #195.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #195