Skip to content
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

Update attention template #30

Merged

Conversation

manupak
Copy link
Contributor

@manupak manupak commented Oct 31, 2024

Ths commit updates the attention
template to include promote operands
and decomposition_config.

Ths commit updates the attention
template to include promote operands
and decomposition_config.

Signed-off-by: Manupa Karunaratne <[email protected]>
Copy link
Contributor

@Groverkss Groverkss left a comment

Choose a reason for hiding this comment

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

Thank you!

@saienduri
Copy link
Contributor

Thanks, can you also change this line https://github.com/nod-ai/iree-kernel-benchmark/blob/main/.github/workflows/run_bench.yml#L19 to mi300-sdxl-kernel, so a runner gets picked up for CI

@manupak
Copy link
Contributor Author

manupak commented Oct 31, 2024

Thanks @saienduri can you "suggest" the change? .. so that I can commit it :). I need to setup ssh on the machine im currently in to change a workflow as PAT does not allow that.

@manupak
Copy link
Contributor Author

manupak commented Oct 31, 2024

nvm let me setup ssh... I dont think you can suggest as it is not changed in the PR.

@saienduri
Copy link
Contributor

Cool, you can also just use the github UI and edit the file on your branch if it's too much trouble

@manupak
Copy link
Contributor Author

manupak commented Oct 31, 2024

@saienduri thanks for the tip! i did not know about GH UI :o

@manupak
Copy link
Contributor Author

manupak commented Nov 1, 2024

@saienduri as first timer here, who should merge the PR :) ?

@saienduri
Copy link
Contributor

Once you get an approval, go for it :). Merging it now though

@saienduri saienduri merged commit 72c2ae3 into nod-ai:main Nov 2, 2024
1 check passed
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