-
Notifications
You must be signed in to change notification settings - Fork 772
{devel}[gfbf/2024a] Triton v3.3.1 w/ CUDA 12.6.0 #23120
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
{devel}[gfbf/2024a] Triton v3.3.1 w/ CUDA 12.6.0 #23120
Conversation
Updated software
|
|
Test report by @Flamefire |
|
Test report by @Flamefire |
|
Test report by @Flamefire |
…s: Triton-3.3.1_disable-git-submodule-call.patch, Triton-3.3.1_disable-git-version-check.patch
1770ef2 to
b8d2bde
Compare
|
Test report by @Flamefire |
|
Test report by @Flamefire |
|
Test report by @Flamefire |
This ensures the `TRITION_*_PATH` variables are set correctly and a basic example works. Also move the lit dependency to the build deps as it is only required for tests.
d978096 to
ec98d61
Compare
|
Test report by @Flamefire |
|
Test report by @Flamefire |
|
Test report by @Flamefire |
|
@Flamefire now that easybuilders/easybuild-framework#4980 is merged, what else is needed here? Or do you think it's ready? |
The other framework PR is required too |
|
Okay. Seems there are conflicts to be resolved in easybuilders/easybuild-easyblocks#3887 |
|
The alternative is to move the postinstall patch as I had at some point: ec98d61 But I like it better this way. I'll look into the conflict |
|
Test report by @pavelToman |
|
Test report by @pavelToman |
|
Test report by @pavelToman |
|
@boegelbot please test @ jsc-zen3-a100 |
|
@boegel: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... - notification for comment with ID 3406069678 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
Test report by @boegel |
easybuild/easyconfigs/t/Triton/Triton-3.3.1-gfbf-2024a-CUDA-12.6.0.eb
Outdated
Show resolved
Hide resolved
98ef576 to
de3bd9e
Compare
|
@boegelbot please test @ jsc-zen3-a100 |
|
@boegel: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... - notification for comment with ID 3409579469 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
Test report by @akesandgren |
|
Test report by @boegel |
|
@akesandgren How did this happen?! |
|
Hmm might have forgotten to update my devel tree before running that... |
|
Test report by @akesandgren |
akesandgren
left a comment
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.
LGTM
|
Going in, thanks @Flamefire! |
(created using
eb --new-pr)This is for PyTorch 2.7
Requires:
CMakeMaketo specify name of build directory easybuild-easyblocks#3786Bundlegeneric easyblock to support use of post-install patches easybuild-easyblocks#3887Alternatively we could put the postinstall patch to the extension if easybuilders/easybuild-framework#4980 gets merged