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

【PIR API adaptor No.170、176】Migrate prior_box/reciprocal into pir #59042

Merged
merged 6 commits into from
Nov 17, 2023

Conversation

ccsuzzh
Copy link
Contributor

@ccsuzzh ccsuzzh commented Nov 15, 2023

PR types

Others

PR changes

APIs

Description

No.170 将 paddle.vision.ops.prior_box 迁移升级至 pir,并更新单测,单测覆盖率:4/4
No.176 将 paddle.Tensor.reciprocal 迁移升级至 pir,并更新单测,单测覆盖率:4/4

@ccsuzzh
Copy link
Contributor Author

ccsuzzh commented Nov 16, 2023

@MarioLulab 研发大哥,为啥我pre-commit在本地没问题,ci却一直报错呢?

@luotao1
Copy link
Contributor

luotao1 commented Nov 16, 2023

@ccsuzzh 本地是最新的develop么?因为昨天 @gouzil 川师傅升级了 isort 版本,可能有影响

@ccsuzzh
Copy link
Contributor Author

ccsuzzh commented Nov 16, 2023

@ccsuzzh 本地是最新的develop么?因为昨天 @gouzil 川师傅升级了 isort 版本,可能有影响

不是最新的,那我更新一下。

@luotao1 luotao1 added contributor External developers HappyOpenSource 快乐开源活动issue与PR labels Nov 16, 2023
Copy link
Contributor

@MarioLulab MarioLulab left a comment

Choose a reason for hiding this comment

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

LGTM

@luotao1 luotao1 merged commit 25313f7 into PaddlePaddle:develop Nov 17, 2023
28 checks passed
@ccsuzzh ccsuzzh deleted the prior_box branch November 20, 2023 00:44
SecretXV pushed a commit to SecretXV/Paddle that referenced this pull request Nov 28, 2023
…ddlePaddle#59042)

* Migrate prior_box/reciprocal into pir

* fix

* fix

* fix bug

* fix codestyle
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers HappyOpenSource 快乐开源活动issue与PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants