-
Notifications
You must be signed in to change notification settings - Fork 5.6k
OP test accuracy requirements (English)
Tao Luo edited this page Dec 16, 2019
·
1 revision
OP Test Specification:The default accuracy values (atol, rtol, eps or max_relative_error) in op tests are not allowed to be modified.
Specification summary:
- Section 1, Explanations Of This Specification
- Section 2, This specification checks the default values of the following parameters which cannot be modified in principle.
- Section 3, CI Method
- Section 4, Suggestions Of Check Failure
Supplementary note:
- You may find some aspects that are not taken into account in the existing specifications, which need to be continuously supplemented and improved during the implementation process. Please feel free to give your feedback.
At present, there is a problem in OP single tests that thresholds of some parameters are enlarged to pass the tests, when checking the accuracy of forward and reverse gradient outputs. To ensure the quality of OPs, this specification is proposed and corresponding inspection methods are added in CI.
Requirement: After setting the default value of parameters in the outputs check functions, it is not allowed to modify the default values in each OP single test.
2. This specification checks the default values of the following parameters which cannot be modified in principle.
- Forward output check
- Single test interface function
check_output
,check_output_with_place
: parameteratol
- Single test interface function
- Reverse gradient check
- Single test interface function
check_grad
,check_grad_with_place
: parametermax_relative_error
- Single test interface function
grad_check
,double_grad_check
: parameterseps
,atol
,rtol
- Single test interface function
- In the addition/modification content of the OP single test files, as long as the above parameter keywords are scanned to modify the default values, CI will fail and report errors.
It is recommended to use the default value of the function. If there are special requirements, it needs to be reviewed by several specific people.
If you encounter other problems, please contact @GaoWei8。