-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Adapting npu for FusedHeadAndCrossEntropy #9499
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
Conversation
|
Thanks for your contribution! |
| logits_chunk_cast_lst, | ||
| logits_chunk_cast, | ||
| group=model_parallel_group, | ||
| if get_env_device() == "npu": |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
封一个函数?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #9499 +/- ##
===========================================
+ Coverage 52.80% 53.02% +0.21%
===========================================
Files 688 701 +13
Lines 109331 110693 +1362
===========================================
+ Hits 57730 58690 +960
- Misses 51601 52003 +402 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
gongweibao
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
wawltor
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* Adapting npu for FusedHeadAndCrossEntropy * wrapper npu function
* Adapting npu for FusedHeadAndCrossEntropy * wrapper npu function
PR types
Others
PR changes
Others
Description
npu 动态切片性能差,使用split和concat来绕过使用动态切片