Skip to content

Remove triton_util, superseded by native Triton APIs#180

Merged
fulvius31 merged 1 commit intoredhat-et:mainfrom
fulvius31:remove-triton-util
Apr 10, 2026
Merged

Remove triton_util, superseded by native Triton APIs#180
fulvius31 merged 1 commit intoredhat-et:mainfrom
fulvius31:remove-triton-util

Conversation

@fulvius31
Copy link
Copy Markdown
Collaborator

@fulvius31 fulvius31 commented Apr 7, 2026

Summary

  • Remove triton_util/ package and all associated files
  • Delete .github/workflows/build-triton-util.yml workflow
  • Clean up triton_util references from Readme.md and pylint.yml

triton_util provided higher-level abstractions for loading/storing
(e.g. load_2d), debugging helpers (print_once, breakpoint_if),
and utilities (cdiv). These are now covered natively by Triton:

triton_util Triton native
load_2d / store_2d make_tensor_descriptor + load_tensor_descriptor (TMA)
print_once / print_if device_print
breakpoint_once / breakpoint_if device_assert + debug_barrier
cdiv tl.cdiv
assert_tensors_gpu_ready Built-in mask/boundary_check on tl.load/tl.store

Summary by CodeRabbit

  • Documentation

    • Removed triton_util documentation and README references from the project docs.
  • Chores

    • Removed triton_util package configuration, packaging entrypoint, tests, and package initializer exports.
    • Removed related CI workflows (build and lint) and test-related CI triggers.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 7, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 36e164a6-20ee-4738-8477-0f1c4f8796c9

📥 Commits

Reviewing files that changed from the base of the PR and between acde522 and 300877a.

📒 Files selected for processing (12)
  • .github/workflows/build-triton-util.yml
  • .github/workflows/pylint.yml
  • Readme.md
  • triton_util/README.md
  • triton_util/setup.py
  • triton_util/tests/conftest.py
  • triton_util/tests/test_coding.py
  • triton_util/tests/test_debugging.py
  • triton_util/tests/test_loading.py
  • triton_util/triton_util/__init__.py
  • triton_util/triton_util/coding.py
  • triton_util/triton_util/debugging.py

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to data retention organization setting


📝 Walkthrough

Walkthrough

This pull request removes the triton_util package and its artifacts: source modules, tests, package config, package docs, and an associated CI workflow; it also narrows pylint CI triggers/config to exclude triton_util.

Changes

Cohort / File(s) Summary
CI Workflows
.github/workflows/build-triton-util.yml, .github/workflows/pylint.yml
Deleted build-triton-util.yml. Updated pylint.yml to stop running on triton_util/** and removed the editable triton_util[dev] install step.
Documentation
Readme.md, triton_util/README.md
Removed the "Triton Util" section from Readme.md and deleted the package README triton_util/README.md.
Package Setup
triton_util/setup.py, triton_util/triton_util/__init__.py
Deleted setup.py (packaging metadata and extras) and removed the package __init__.py that re-exported submodules.
Source Code
triton_util/triton_util/coding.py, triton_util/triton_util/debugging.py
Removed coding.py (offset/mask/load/store helpers) and debugging.py (PID rule engine, print/breakpoint helpers, tensor readiness assertions, pointer utils).
Tests
triton_util/tests/conftest.py, triton_util/tests/test_coding.py, triton_util/tests/test_debugging.py, triton_util/tests/test_loading.py
Deleted pytest fixture and all test modules covering coding/debugging utilities and interpreter-mode checks.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and directly summarizes the main change: removal of the triton_util package, which is justified by native Triton API support.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@fulvius31 fulvius31 force-pushed the remove-triton-util branch from acde522 to 300877a Compare April 10, 2026 13:01
@fulvius31 fulvius31 merged commit d1ed09f into redhat-et:main Apr 10, 2026
3 of 4 checks passed
@fulvius31 fulvius31 deleted the remove-triton-util branch April 10, 2026 13: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