Skip to content

【PaddlePaddle Hackathon 3】Add Paddle ceil operator #12368

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Sep 29, 2022
Merged

【PaddlePaddle Hackathon 3】Add Paddle ceil operator #12368

merged 6 commits into from
Sep 29, 2022

Conversation

@Patrick-Star125
Copy link
Contributor

@Patrick-Star125 Patrick-Star125 commented Jul 30, 2022 •
edited
Loading

@Patrick-Star125 Patrick-Star125 requested a review from a team as a code owner July 30, 2022 05:06
@Ligoml Ligoml mentioned this pull request Aug 1, 2022
Closed
@openvino-dev-samples
Copy link
Contributor

openvino-dev-samples commented Aug 1, 2022 •
edited
Loading

Patrick-Star125

Please update your reference of ceil: which paddle's ceil you try to map and also its introduction link?
It will help to speed-up our further review.

@Patrick-Star125
Copy link
Contributor Author

Patrick-Star125 commented Aug 3, 2022

Reference updated

@yuxu42 yuxu42 requested a review from luo-cheng2021 August 4, 2022 05:21
@yuxu42
Copy link
Contributor

yuxu42 commented Aug 4, 2022

@luo-cheng2021 Could you please review?

@Patrick-Star125
Copy link
Contributor Author

Patrick-Star125 commented Aug 27, 2022

These CI tests failed and I can't check the Details
image

@Patrick-Star125 Patrick-Star125 requested a review from a team as a code owner August 27, 2022 07:24
@ilyachur ilyachur added this to the 2022.3 milestone Sep 1, 2022
@ilyachur ilyachur added the category: PDPD FE OpenVINO PaddlePaddle FrontEnd label Sep 1, 2022
ceciliapeng2011
ceciliapeng2011 approved these changes Sep 1, 2022
View reviewed changes
Copy link
Contributor

@ceciliapeng2011 ceciliapeng2011 left a comment

Choose a reason for hiding this comment

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

LGTM

@ceciliapeng2011 ceciliapeng2011 added the PaddlePaddle Hackathon a Intel and Baidu joint Hackathon event label Sep 1, 2022
@ilyachur ilyachur enabled auto-merge (squash) September 13, 2022 12:02
@Patrick-Star125
Copy link
Contributor Author

Patrick-Star125 commented Sep 20, 2022

this PR will merge automatically when all requirements are met, Is this mean I need to merge master to this branch and make it newest status? or it just require that all CI pass?

@ilyachur ilyachur merged commit fbd53ec into openvinotoolkit:master Sep 29, 2022
@ilya-lavrenov ilya-lavrenov added the ExternalPR External contributor label Mar 4, 2023
@xdxd1234-bit
Copy link

xdxd1234-bit commented Nov 25, 2023

请问这个报错的原因是什么呢 我第一次接触为openvino添加算子还不是太懂
e184fb59382fadc1c821d0c6f0e0405

@openvino-dev-samples
Copy link
Contributor

openvino-dev-samples commented Nov 27, 2023

请问这个报错的原因是什么呢 我第一次接触为openvino添加算子还不是太懂 e184fb59382fadc1c821d0c6f0e0405

你看下 ./test_model_zoo/paddle_test_models/ 这个路径下是否有模型文件？

@xdxd1234-bit
Copy link

xdxd1234-bit commented Dec 4, 2023

bin/intel64/Release/test_model_zoo下没有paddle_test_models只有core和onnx

@openvino-dev-samples
Copy link
Contributor

openvino-dev-samples commented Dec 4, 2023

你看下有没有安装paddle的环境，如果没有的话，就不会生成测试用例

@xdxd1234-bit
Copy link

xdxd1234-bit commented Dec 4, 2023

感谢您的回复，那我需要安装完paddle以后，在重新对openvino进行源码编译吗

@openvino-dev-samples
Copy link
Contributor

openvino-dev-samples commented Dec 4, 2023

感谢您的回复，那我需要安装完paddle以后，在重新对openvino进行源码编译吗

是的

@xdxd1234-bit
Copy link

xdxd1234-bit commented Dec 4, 2023

不好意思麻烦您了 这个paddle和openvino之间的版本对应有要求吗，我安装完paddle以后 重新编译还是没有padle_test我安装paddle的版本是
微信图片_20231204145636
openvino是直接git clone的

@openvino-dev-samples
Copy link
Contributor

openvino-dev-samples commented Dec 4, 2023

不好意思麻烦您了 这个paddle和openvino之间的版本对应有要求吗，我安装完paddle以后 重新编译还是没有padle_test我安装paddle的版本是 微信图片_20231204145636 openvino是直接git clone的

编译过程中会自动安装paddle，如果没有的话，最好装Paddle 2.4 CPU版本的，然后重新编译openvino

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ceciliapeng2011 ceciliapeng2011 ceciliapeng2011 approved these changes

@luo-cheng2021 luo-cheng2021 luo-cheng2021 approved these changes

+1 more reviewer

@ilyachur ilyachur ilyachur approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

2022.3

Development

Successfully merging this pull request may close these issues.

8 participants

@Patrick-Star125 @openvino-dev-samples @yuxu42 @xdxd1234-bit @ilyachur @ceciliapeng2011 @luo-cheng2021 @ilya-lavrenov