-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
【Hackathon 5th No.96】add paddle unstack op #20080
【Hackathon 5th No.96】add paddle unstack op #20080
Conversation
auto data = node.get_input("X"); | ||
auto dim = node.get_attribute<int32_t>("axis"); | ||
if (dim < 0) { | ||
dim = dim + data.get_partial_shape().rank().get_length(); |
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.
Can we remove this line, because split op can accept negative axis: https://github.com/openvinotoolkit/openvino/blob/master/docs/articles_en/documentation/openvino_ir/operation_sets/operations_specifications/movement/Split_1.md
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.
done
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.
have you validated it again? Could you help to update your uni-test screenshot if not ?
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.
sorry, missed the notification email, we should keep these code because index access of shape
is bounded within [0, abs(dim)-1]
@yuxu42 Could you help to have a review on this PR, thanks |
@liubo-intel Could you please help review? Thanks! |
Done |
Hi, @Patrick-Star125 : please fix the clang-format issue by following suggestion of the CI log. |
@ceciliapeng2011 can we merge the PR? Thanks! |
* add paddle mapping * develop test sampes * remove redundant code * revoke code change * add type mapping * add test sample * format code --------- Co-authored-by: Your Name <you@example.com>
Details:
add unstack operation in Paddle front end
Reference:
Requirements
unit-test