Skip to content
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

【PaddlePaddle Hackathon 4】add paddle grid_sampler op #15869

Merged
merged 21 commits into from
May 12, 2023
Merged

【PaddlePaddle Hackathon 4】add paddle grid_sampler op #15869

merged 21 commits into from
May 12, 2023

Conversation

@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 xczhai March 6, 2023 03:56
@OpenVINO-dev-contest
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

@xczhai
Copy link
Contributor

xczhai 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
51 checks passed
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.

None yet

6 participants