Skip to content

Conversation

@xuanzhang816
Copy link
Contributor

@xuanzhang816 xuanzhang816 commented Aug 29, 2025

[ghstack-poisoned]
xuanzhang816 added a commit that referenced this pull request Aug 29, 2025
ghstack-source-id: bcfd0fe
Pull Request resolved: #527
@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Aug 29, 2025
xuanzhang816 added a commit that referenced this pull request Aug 29, 2025
ghstack-source-id: b085551
Pull Request resolved: #527
xuanzhang816 added a commit that referenced this pull request Aug 29, 2025
ghstack-source-id: 3575edf
Pull Request resolved: #527
@xuanzhang816 xuanzhang816 changed the title jagged hstu example Add jagged hstu attention example Aug 29, 2025
@xuanzhang816 xuanzhang816 requested a review from yf225 August 29, 2025 19:55
)


def tritonbench_hstu_attention_function(inputs: dict[str, Any]) -> torch.Tensor:
Copy link
Contributor

Choose a reason for hiding this comment

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

Curious does it pass accuracy check compared to ragged_attention in tritonbench? If so, maybe we can add a dispatch in KERNEL_MAPPINGS in benchmarks/run.py to integrate with tritonbench.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done :) Could you check and make sure that I added things to KERNEL_MAPPINGS correctly?

@@ -0,0 +1,291 @@
"""
Copy link
Contributor

Choose a reason for hiding this comment

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

Would be great to add a test in test_examples.py too. Thanks!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added this one as well. Let me if I missed something.

Copy link
Contributor

@yf225 yf225 left a comment

Choose a reason for hiding this comment

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

thanks! left some inline comments above

xuanzhang816 added a commit that referenced this pull request Sep 2, 2025
ghstack-source-id: 0be580b
Pull Request resolved: #527
xuanzhang816 added a commit that referenced this pull request Sep 2, 2025
ghstack-source-id: e114db1
Pull Request resolved: #527
xuanzhang816 added a commit that referenced this pull request Sep 2, 2025
ghstack-source-id: 63afb21
Pull Request resolved: #527
@xuanzhang816 xuanzhang816 requested a review from yf225 September 2, 2025 21:44
xuanzhang816 added a commit that referenced this pull request Sep 3, 2025
ghstack-source-id: 3cc18eb
Pull Request resolved: #527
xuanzhang816 added a commit that referenced this pull request Sep 3, 2025
ghstack-source-id: c0da2a9
Pull Request resolved: #527
xuanzhang816 added a commit that referenced this pull request Sep 3, 2025
ghstack-source-id: 3ece772
Pull Request resolved: #527
@yf225 yf225 changed the title Add jagged hstu attention example Add jagged hstu attention example (i.e. ragged_attention) Sep 3, 2025
xuanzhang816 added a commit that referenced this pull request Sep 3, 2025
ghstack-source-id: 5bdb6d0
Pull Request resolved: #527
@xuanzhang816 xuanzhang816 requested a review from yf225 September 3, 2025 19:14
Copy link
Contributor

@yf225 yf225 left a comment

Choose a reason for hiding this comment

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

Thanks so much!

@jansel jansel merged commit ec6099b into gh/xuanzhang816/2/base Sep 3, 2025
13 checks passed
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.

4 participants