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

[Fix](bangc-ops): fix focal_loss_sigmoid_backward gamma range precheck #705

Merged
merged 1 commit into from
Jun 1, 2023

Conversation

stulai
Copy link
Collaborator

@stulai stulai commented Jun 1, 2023

Thanks for your contribution and we appreciate it a lot.

1. Motivation

fix focal_loss_sigmoid_backward gamma range precheck

2. Modification

modified:   kernels/focal_loss_sigmoid/focal_loss_sigmoid.cpp
modified:   mlu_op.h

3. Test Report

(1) gamma test
[2023-6-1 15:37:59] [MLUOP] [Error]:[mluOpFocalLossSigmoidBackward]: gamma should be in the range of [0, 10000]. But now gamma is 20000.
[2023-6-1 15:37:59] [MLUOP] [Error]:"MLUOP_STATUS_NOT_SUPPORTED in mluOpFocalLossSigmoidBackward( handle_, prefer, reduction, input_desc, input_mlu, target_desc, target_mlu, weight_desc, weight_mlu, alpha, gamma, grad_input_desc, grad_input_mlu)"

(2) release_temp cases test
[^ OK ] /MLU_OPS/SOFT_TRAIN/release_temp/focal_loss_sigmoid_backward/focal_loss_sigmoid_backward_data_included_int32_float16_1682494108948.pb
[ OK ] focal_loss_sigmoid_backward/TestSuite.mluOp/1029 (42 ms)
[----------] 1030 tests from focal_loss_sigmoid_backward/TestSuite (114205 ms total)

[----------] Global test environment tear-down
[2023-6-1 15:46:15] [MLUOP] [Vlog]:TearDown CNRT environment.
[ SUMMARY ] Total 1030 cases of 1 op(s).
ALL PASSED.
[==========] 1030 test cases from 1 test suite ran. (114730 ms total)
[ PASSED ] 1030 test cases.

YOU HAVE 7 DISABLED TESTS

@PetrelYy PetrelYy added the BANGC label Jun 1, 2023
@duzekunKTH duzekunKTH merged commit 5afd819 into Cambricon:master Jun 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants