-
Notifications
You must be signed in to change notification settings - Fork 3.1k
【Performance Optimization】Replace cross_entropy_with_softmax to c_softmax_with_cross_entropy in dynamic auto mode #10471
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! |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #10471 +/- ##
========================================
Coverage 48.67% 48.67%
========================================
Files 768 768
Lines 126915 126915
========================================
+ Hits 61777 61778 +1
+ Misses 65138 65137 -1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
7755c5a
to
79970fb
Compare
@@ -26,6 +26,7 @@ | |||
import paddle | |||
import paddle.distributed as dist | |||
from paddle.distributed import fleet | |||
from paddle import in_dynamic_mode |
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
import operator | ||
from typing import Literal, TypeAlias | ||
import paddle.distributed as dist | ||
|
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.
这个文件会长期存在吗,放这个位置合适不? pre-commit好像没过去,应该需要copyright之类的
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
Before submitting
tests
folder. If there are codecov issues, please add tests cases first.PR types
Performance optimization
PR changes
Others
Description
Replace cross_entropy_with_softmax to c_softmax_with_cross_entropy in dynamic auto mode