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

【Hackathon 5th No.97】add paddle tanh_shrink op #20067

Merged
merged 18 commits into from
Dec 12, 2023

Conversation

@Asthestarsfalll Asthestarsfalll requested a review from a team as a code owner September 27, 2023 02:36
@github-actions github-actions bot added category: build OpenVINO cmake script / infra category: PDPD FE OpenVINO PaddlePaddle FrontEnd labels Sep 27, 2023
@github-actions github-actions bot removed the category: build OpenVINO cmake script / infra label Sep 27, 2023
@Asthestarsfalll Asthestarsfalll changed the title TEST 【PaddlePaddle Hackathon 5】add paddle tanh_shrink op Sep 28, 2023
@Asthestarsfalll Asthestarsfalll changed the title 【PaddlePaddle Hackathon 5】add paddle tanh_shrink op 【PaddlePaddle Hackathon No.97】add paddle tanh_shrink op Sep 28, 2023
@Asthestarsfalll Asthestarsfalll changed the title 【PaddlePaddle Hackathon No.97】add paddle tanh_shrink op 【Hackathon 5th No.97】add paddle tanh_shrink op Sep 28, 2023
@openvino-dev-samples
Copy link
Contributor

@yuxu42 Could you help to have a review on this PR, thanks

@yuxu42
Copy link
Contributor

yuxu42 commented Nov 6, 2023

@xczhai Could you please help review? Thanks!

@yuxu42 yuxu42 requested a review from xczhai November 6, 2023 04:51
data = np.random.uniform(-10, 20, [4, 3, 2]).astype(np.float32)
tanh_shrink("tanh_shrink_2", data)

# data = np.random.uniform(10, 20, [2, 3, 4]).astype(np.float16) // only on gpu
Copy link
Contributor

Choose a reason for hiding this comment

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

please remove the useless codes if unnecessary.

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

Copy link
Contributor

@xczhai xczhai left a comment

Choose a reason for hiding this comment

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

LGTM

@xczhai
Copy link
Contributor

xczhai commented Nov 8, 2023

@Asthestarsfalll please rebase the PR. triger the CI again. Thanks.

@yuxu42 yuxu42 added PaddlePaddle Hackathon a Intel and Baidu joint Hackathon event ExternalPR External contributor labels Nov 10, 2023
@xczhai xczhai enabled auto-merge (squash) November 13, 2023 05:39
@xczhai
Copy link
Contributor

xczhai commented Nov 21, 2023

@ceciliapeng2011 Hi Cecilia, could you help merge the PR? Thanks.

@yuxu42
Copy link
Contributor

yuxu42 commented Dec 4, 2023

@Asthestarsfalll could you rebase and fix the CI issue? Thanks!

@xczhai xczhai merged commit e51ba62 into openvinotoolkit:master Dec 12, 2023
72 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.

5 participants