Skip to content

Conversation

@Patrick-Star125
Copy link
Contributor

@Patrick-Star125 Patrick-Star125 commented Feb 22, 2023

@Patrick-Star125 Patrick-Star125 requested a review from a team as a code owner February 22, 2023 03:56
@github-actions github-actions bot added the category: PDPD FE OpenVINO PaddlePaddle FrontEnd label Feb 22, 2023
@Patrick-Star125 Patrick-Star125 changed the title 【PaddlePaddle Hackathon 4】add paddle grip_sample op 【PaddlePaddle Hackathon 4】add paddle grid_sample op Feb 24, 2023
@Patrick-Star125 Patrick-Star125 changed the title 【PaddlePaddle Hackathon 4】add paddle grid_sample op 【PaddlePaddle Hackathon 4】add paddle grid_sampler op Mar 1, 2023
@Patrick-Star125
Copy link
Contributor Author

Patrick-Star125 commented Mar 1, 2023

@OpenVINO-dev-contest could you review this pr please?

@ilya-lavrenov ilya-lavrenov added ExternalPR External contributor PaddlePaddle Hackathon a Intel and Baidu joint Hackathon event labels Mar 4, 2023
@yuxu42 yuxu42 requested a review from azhai219 March 6, 2023 03:56
@openvino-dev-samples
Copy link
Contributor

@OpenVINO-dev-contest could you review this pr please?

Hi, Im ok with the implementation, Did you refer to openvino's onnx frontend ? if yes, pls add the link into this conversion and it will be helpful for other reviewer to understand your solution more easily. thanks

@Patrick-Star125
Copy link
Contributor Author

@xczhai

  1. the default value of align_corners should be true according to paddle source code
  2. test sample without optional inputs has been added
  3. test samples with distinct shapes had been added

@azhai219
Copy link
Contributor

azhai219 commented Mar 10, 2023

@xczhai

  1. the default value of align_corners should be true according to paddle source code
  2. test sample without optional inputs has been added
  3. test samples with distinct shapes had been added

Hi @Patrick-Star125 ,

@Patrick-Star125
Copy link
Contributor Author

Got it, test sample with dynami shape has been added.

auto data = node.get_input("X");
auto grid = node.get_input("Grid");
default_opset::GridSample::Attributes attributes{};

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please correct the code format here as log reports.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@openvino-comment-bot
Copy link

Can one of the admins verify this patch?

@ceciliapeng2011 ceciliapeng2011 merged commit 306df51 into openvinotoolkit:master May 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: PDPD FE OpenVINO PaddlePaddle FrontEnd ExternalPR External contributor PaddlePaddle Hackathon a Intel and Baidu joint Hackathon event

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants