-
-
Notifications
You must be signed in to change notification settings - Fork 16.6k
python3Packages.triton: 3.3.1 -> 3.4.0 #432046
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
python3Packages.triton: 3.3.1 -> 3.4.0 #432046
Conversation
66452e4
to
e044761
Compare
e044761
to
a9b8b2c
Compare
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.
Diff lgtm, traveling right now so hard to test. Let me know if it doesn't build and I can take a look.
@@ -65,7 +65,7 @@ let | |||
in | |||
stdenv.mkDerivation (finalAttrs: { | |||
pname = "triton-llvm"; | |||
version = "21.0.0-git"; # See https://github.com/llvm/llvm-project/blob/main/cmake/Modules/LLVMVersion.cmake | |||
version = "21.0.0-unstable-2025-06-10"; # See https://github.com/llvm/llvm-project/blob/main/cmake/Modules/LLVMVersion.cmake |
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.
Nice catch, makes sense to me (I tunnel visioned too much on the llvm tag and forgot about nixpkgs's conventions).
and does #431973 depend on this PR? In my experience newer versions of triton are often not backwards compatible with older versions so triton will have to be bumped simultaneously with torch, or is this not true? |
552d693
to
477d9f5
Compare
477d9f5
to
78a46c0
Compare
After 4+ hours of compilation, I can confirm that |
|
|
@kirillrdy this one should be ready to go too :) |
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.
tried building but gave up waiting
This breaks torch.compile triton (maybe only on ROCm)!
Reportedly triton 3.4 needs torch 2.8+ Full error I'm hitting on a tiny modded-nanogpt train which occurs when it tries to autotune flex_attn after applying this upgrade:
|
Thanks for reporting. I'm working on packaging |
Will see if we can backport compat for triton 2.4 in the short term since the 2.8 bump looks a bit complicated. |
Things done
Diff: triton-lang/triton@v3.3.1...v3.4.0
cc @SomeoneSerge @Madouura @DerDennisOP
passthru.tests
.nixpkgs-review
on this PR. See nixpkgs-review usage../result/bin/
.Add a 👍 reaction to pull requests you find important.