-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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.51、175】 Migrate paddle.vision.ops.decode_jpeg,paddle.vision.ops.read_file into pir #58955
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
read_file op 需要迁移到 PHI 体系下,内部已经记录该问题。这个 pr 可以先搁置 |
收到 |
Sorry to inform you that 9ad3f7f's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually. |
|
test/legacy_test/test_read_file.py
Outdated
@@ -36,35 +38,47 @@ def tearDown(self): | |||
def read_file_decode_jpeg(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个函数改成test_开头吧,不然单测不会执行,也就导致覆盖率的ci没有过~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
收到
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
test/legacy_test/test_read_file.py
Outdated
|
||
def test_read_file_decode_jpeg_dynamic(self): | ||
self.read_file_decode_jpeg() | ||
class TestReadFileWithStatic(unittest.TestLoader): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
class TestReadFileWithStatic(unittest.TestLoader): | |
class TestReadFileWithStatic(unittest.TestCase): |
设置错了的话单测不会运行,导致 ci-coverage 不通过
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
收到
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…dle.vision.ops.read_file into pir (PaddlePaddle#58955)
PR types
Others
PR changes
APIs
Description
No.51 paddle.vision.ops.decode_jpeg
No.175 paddle.vision.ops.read_file
PIR API 推全升级
将
paddle.vision.ops.decode_jpeg
和paddle.vision.ops.read_file
迁移升级至 pir,并更新单测 单测覆盖率:1/1