Skip to content

Respect $TRITON_HOME#7907

Merged
tohtana merged 1 commit intodeepspeedai:masterfrom
Flamefire:tthome
Mar 23, 2026
Merged

Respect $TRITON_HOME#7907
tohtana merged 1 commit intodeepspeedai:masterfrom
Flamefire:tthome

Conversation

@Flamefire
Copy link
Copy Markdown
Contributor

The default directory for Triton is only ~/.triton if $TRITON_HOME is not set or empty.

Although TRITON_CACHE_DIR is already supported (which deviates from ~/.triton/cache as used by Triton) users might expect that the root dir variable is also supported to avoid having to set all TRITON_* variables

@Flamefire Flamefire requested a review from tohtana as a code owner March 16, 2026 13:09
The default directory for Triton is only `~/.triton`
if `$TRITON_HOME` is not set or empty.

Signed-off-by: Alexander Grund <alexander.grund@tu-dresden.de>
@tohtana tohtana added this pull request to the merge queue Mar 16, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Mar 16, 2026
@tohtana tohtana added this pull request to the merge queue Mar 18, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Mar 18, 2026
@tohtana tohtana merged commit 38bd11a into deepspeedai:master Mar 23, 2026
9 checks passed
@Flamefire Flamefire deleted the tthome branch March 23, 2026 10:26
nathon-lee pushed a commit to nathon-lee/DeepSpeed_woo that referenced this pull request Mar 28, 2026
The default directory for Triton is only `~/.triton` if `$TRITON_HOME`
is not set or empty.

Although `TRITON_CACHE_DIR` is already supported (which deviates from
`~/.triton/cache` as used by Triton) users might expect that the root
dir variable is also supported to avoid having to set all `TRITON_*`
variables

Signed-off-by: Alexander Grund <alexander.grund@tu-dresden.de>
Signed-off-by: nathon-lee <leejianwoo@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants