-
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.284, 293】 Migrate paddle.Tensor.nanquantile and paddle.Tensor.quantile into pir #59572
【PIR API adaptor No.284, 293】 Migrate paddle.Tensor.nanquantile and paddle.Tensor.quantile into pir #59572
Conversation
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~
可以在 pr 描述里写一下:
- nanquantile 在
test/legacy_test/test_zero_dim_tensor.py
里的 test_nanquantile 单测暂不支持。因为 paddle.static.append_backward 尚未支持 pir 模式 - quantile 在
test/legacy_test/test_zero_dim_tensor.py
里的 TestSundryAPIStatic.test_quantile 单测暂不支持。因为 paddle.static.append_backward 尚未支持 pir 模式
然后分别更新一下 nanquantile 和 quantile 的单测覆盖率
已修改,感谢~ |
代码冲突了,辛苦解决下冲突~ |
d2c6716
to
8d28aad
Compare
不好意思之前忘记了,已解决冲突,麻烦再review一下~ |
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.
nice work ~ 有些小地方需要修改
还需要 pre-commit 一下
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
PR types
Others
PR changes
APIs
Description
PIR API 推全升级
#58067
No.284 将 paddle.Tensor.nanquantile迁移升级至 pir,并更新单测,单测覆盖率:1/2。
nanquantile 在 test/legacy_test/test_zero_dim_tensor.py 里的 test_nanquantile 单测暂不支持。因为 paddle.static.append_backward 尚未支持 pir 模式
No.293 将 paddle.Tensor.quantile迁移升级至 pir,并更新单测,单测覆盖率:1/2。
quantile 在 test/legacy_test/test_zero_dim_tensor.py 里的 TestSundryAPIStatic.test_quantile 单测暂不支持。因为 paddle.static.append_backward 尚未支持 pir 模式