Skip to content

Add launch hooks #15

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Add launch hooks #15

wants to merge 4 commits into from

Conversation

FindHao
Copy link
Member

@FindHao FindHao commented Jun 16, 2025

Summary:

  • fix lint for extract_file_content.
  • Introduce add_launch_metadata function and JITHookImpl and LaunchHookImpl classes to manage launch metadata and hooks in Triton.
  • Update init function to register the new hooks for JIT compilation and launch events.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label Jun 16, 2025
FindHao added 4 commits June 17, 2025 15:45
Summary:
- Refactor file size check and error handling in `extract_file_content` for improved readability.
- Introduce `add_launch_metadata` function and `JITHookImpl` and `LaunchHookImpl` classes to manage launch metadata and hooks in Triton.
- Update `init` function to register the new hooks for JIT compilation and launch events.
Summary:
- Refactor `convert` function to handle additional data types including NaN/Infinity for floats, datetime, enums, and paths.
- Introduce `extract_arg_info` function to extract detailed information from kernel arguments, particularly for PyTorch tensors.
- Update `add_launch_metadata` to include extracted argument information in the launch metadata.
- Improve logging for unknown types in the `convert` function.

This update improves the structured logging capabilities, providing more comprehensive insights into kernel arguments and their properties.
@FindHao FindHao force-pushed the findhao/add_launch_hook branch from 63f6c80 to 2ab81b9 Compare June 17, 2025 22:46
@FindHao FindHao marked this pull request as ready for review June 17, 2025 22:46
@facebook-github-bot
Copy link
Contributor

@FindHao has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Meta Open Source bot.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants