Skip to content

Fix tt.dot to support integers#170

Merged
nhat-nguyen merged 5 commits intomicrosoft:mainfrom
parsifal-47:fix-dot
Aug 29, 2024
Merged

Fix tt.dot to support integers#170
nhat-nguyen merged 5 commits intomicrosoft:mainfrom
parsifal-47:fix-dot

Conversation

@parsifal-47
Copy link
Copy Markdown
Contributor

Core tests are passing with current triton code, but this change:
triton-lang/triton#4573
should uncover more testcases, I checked that updates tests are also passing.

Comment thread include/triton-shared/Conversion/TritonArithToLinalg/ConversionPatterns.hpp Outdated
@parsifal-47
Copy link
Copy Markdown
Contributor Author

I have noticed the failure in lit tests, will follow-up with the fix

@parsifal-47
Copy link
Copy Markdown
Contributor Author

Lit test should be good now

@parsifal-47
Copy link
Copy Markdown
Contributor Author

I saw: "AttributeError: 'torch.dtype' object has no attribute 'startswith'", but it works for me, looks like I have a different version of pytorch, updated the code

@parsifal-47
Copy link
Copy Markdown
Contributor Author

failed again on bfloa16, should be fixed now

@nhat-nguyen nhat-nguyen merged commit 989999a into microsoft:main Aug 29, 2024
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