-
Notifications
You must be signed in to change notification settings - Fork 220
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
Implement SoftMarginLoss #3226
base: develop
Are you sure you want to change the base?
Implement SoftMarginLoss #3226
Conversation
- remove duplicate api in miopen.h - add MIOPEN_INTERNALS_EXPORT - change how to get env in gtest
@CAHEK7 this PR has a very similar code structure to this MultiMarginLoss PR you have reviewed before. |
src/kernels/MIOpenSoftMarginLoss.cpp
Outdated
{ | ||
// If reduction = None, O is DTYPE* | ||
case 0: static_cast<DTYPE*>(O)[O_tv.get_tensor_view_idx(idx)] = CVT_ACCUM2FLOAT(loss); break; | ||
// If reduction != Sum, O is FLOAT_ACCUM* and then all elements will be sum up in the next |
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.
I think the cmt should be If reduction = Sum
, and the same with case 2
.
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.
The PR is blocked by the following errors caused by the CI issue we recently have. After the CI issue is resolved, this PR need to restart to run through the CI.
[2024-09-24T07:18:43.885Z] Exception occurred: org.kohsuke.github.HttpException: {"message":"API rate limit exceeded for user ID 49319081. If you reach out to GitHub Support for help, please include the request ID
@littlecutebird the PR has some build issues, could you follow up with the fix? Thanks! |
Unreduced:
fp32 forward
fp32 backward
fp16 forward
fp16 backward
bfp16 forward
bfp16 backward
Reduced:
fp32 forward
fp32 backward
fp16 forward
fp16 backward
bfp16 forward
bfp16 backward