Skip to content

[WIP] Try new Triton package 20260608#392

Closed
antiagainst wants to merge 1 commit into
mainfrom
lei/try-triton
Closed

[WIP] Try new Triton package 20260608#392
antiagainst wants to merge 1 commit into
mainfrom
lei/try-triton

Conversation

@antiagainst

Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Lei Zhang <antiagainst@gmail.com>
@antiagainst antiagainst requested a review from a team as a code owner June 9, 2026 03:59

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6dc5712a68

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment on lines +5 to +6
tokenspeed-proton==3.7.10.post20260608
tokenspeed-triton==3.7.10.post20260608

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Keep staged pins out of release requirements

When this exact post20260608 pin is still only available from TestPyPI, the tokenspeed-kernel release build has no way to resolve it: .github/workflows/release-tokenspeed-kernel.yml builds wheels with only the PyTorch extra index, and setup.py's build path installs requirements/cuda.txt, which includes this file. The new CI install script preinstalls from TestPyPI only for test/ci_system/install_deps.sh, so release wheels and any source install outside that script fail before the kernel build starts. Either make the release/source build use the same index explicitly or avoid committing TestPyPI-only exact pins in the shared requirements.

Useful? React with 👍 / 👎.

@lightseek-bot

Copy link
Copy Markdown
Contributor

will we bump the stable version @antiagainst

@lightseek-bot lightseek-bot deleted the lei/try-triton branch June 13, 2026 01:02
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.

2 participants