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.

Sign up for GitHub

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

Jump to bottom

[NPU] Fix ce loss in npu deveice #3759

Merged
merged 2 commits into from
Jul 24, 2024
Merged

[NPU] Fix ce loss in npu deveice #3759

merged 2 commits into from
Jul 24, 2024

Conversation

Birdylx
Copy link
Collaborator

@Birdylx Birdylx commented Jul 24, 2024

PR types

PR changes

Description

@paddle-bot Paddle-bot
Copy link

paddle-bot bot commented Jul 24, 2024

Thanks for your contribution!

@Birdylx Birdylx requested a review from nepeplwu July 24, 2024 09:24
nepeplwu
nepeplwu approved these changes Jul 24, 2024
View reviewed changes
Copy link
Collaborator

@nepeplwu nepeplwu left a comment

Choose a reason for hiding this comment

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

LGTM

@Birdylx Birdylx merged commit df23cf2 into PaddlePaddle:develop Jul 24, 2024
3 checks passed
@Birdylx Birdylx deleted the fix/ce_loss branch July 24, 2024 09:41
Birdylx added a commit to Birdylx/PaddleSeg that referenced this pull request Aug 21, 2024
@Birdylx
replace ce loss with nll loss (PaddlePaddle#3759)
823d3e3
Birdylx added a commit to Birdylx/PaddleSeg that referenced this pull request Aug 21, 2024
@Birdylx
replace ce loss with nll loss (PaddlePaddle#3759)
4c8d180
Birdylx added a commit that referenced this pull request Aug 23, 2024
@Birdylx
[NPU] replace ce loss with nll loss (#3759) (#3782)
4b37151
Birdylx added a commit that referenced this pull request Aug 23, 2024
@Birdylx
[NPU][Cherry-pick] Pick #3717, #3727, #3759 (#3783)
f881ddf 
* Add npu support (#3717)

* [NPU] Fix nan loss and speed up ce loss (#3727)

* Fix nan loss

* Speed up ce loss

* replace ce loss with nll loss (#3759)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nepeplwu nepeplwu nepeplwu approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Birdylx @nepeplwu