-
Notifications
You must be signed in to change notification settings - Fork 53
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
转换规则 No.86 #202
转换规则 No.86 #202
Conversation
Thanks for your contribution! |
paconvert/api_matcher.py
Outdated
paddle_class = self.paddleClass | ||
code = "{}.dtype not in [paddle.uint8]".format(paddle_class) | ||
node = ast.parse(code.strip("\n")).body | ||
return node |
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.
return ast.parse(code.strip("\n")).body
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.
已修改
paconvert/api_mapping.json
Outdated
"torch.Tensor.is_signed": {}, | ||
"torch.Tensor.is_signed": { | ||
"Matcher": "TensorIsSignedMatcher", | ||
"paddle_api": "paddle.Tensor.dtype" |
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.
组合实现也不用填paddle_api了,与单个API的差异比较大,依赖多个组合,因此无需填写
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.
已修改
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.
LGTM
PR Docs
#112
086 torch.Tensor.is_signed PaddlePaddle/docs#6054
PR APIs