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

【Hackathon 6th Fundable Projects 3 No.227】 [fluid_ops] matmul #65939

Merged
merged 4 commits into from
Sep 6, 2024

Conversation

co63oc
Copy link
Contributor

@co63oc co63oc commented Jul 11, 2024

PR Category

Operator Mechanism

PR Types

Others

Description

迁移 matmul 为 legacy_matmul

test/legacy_test/test_matmul_op_with_head.py 单测使用的是 matmul 增加multihead 参数,已有 multihead_matmul 算子,删除multihead单测

scale onednn kernel增加类型 int8_t, uint8_t

@paddle-bot paddle-bot bot added the contributor External developers label Jul 11, 2024
@co63oc co63oc force-pushed the fix_matmul branch 4 times, most recently from dd232a9 to 17d7ae5 Compare July 21, 2024 05:39
Copy link

paddle-ci-bot bot commented Aug 5, 2024

Sorry to inform you that b1dfd96's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually.

@co63oc co63oc force-pushed the fix_matmul branch 6 times, most recently from b417800 to 62d1dc7 Compare August 30, 2024 12:56
@co63oc co63oc changed the title CI测试不review matmul [fluid_ops] 【Hackathon 6th Fundable Projects 3 No.227】 [fluid_ops] matmul Aug 31, 2024
Copy link
Contributor

@zyfncg zyfncg left a comment

Choose a reason for hiding this comment

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

image

这个反向kernel注册类型的问题看怎么加一下

@co63oc
Copy link
Contributor Author

co63oc commented Sep 4, 2024

image
已修改反向不支持为int8_t类型,fluid 和 phi中反向都不支持int8_t类型
fluid 支持类型
image
phi 支持类型
image

@luotao1 luotao1 merged commit a2e7730 into PaddlePaddle:develop Sep 6, 2024
28 of 29 checks passed
@co63oc co63oc deleted the fix_matmul branch September 20, 2024 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants