fix the XXX_GRAD_CASE bug by HexToString #32004
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Thanks for your contribution! |
|
|
Superjomn
reviewed
Apr 2, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR types
Bug fixes
PR changes
OPs
Describe
修复厂内用户发现的expand_v2、tile两个OP反向梯度求解，当shape维度为6时，返回梯度信息为None的bug
代码搜查同时发现其他四个OP:expand、expand_as、expand_as_v2、meshgrid存在同样问题一并修复。
问题原因：
BOOST_PP_REPEAT()是个高级宏，会生成从0-(N-1)的传入的下面的宏，当传入n=6时候，宏展开生成的代码中包含0-5的case
而代码中，switch-case的判断变量dims取的是size，取值范围为1-6，故当取6时候，找不到对应的case.