-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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 test_errors 相关单测适配 #60696
Labels
HappyOpenSource
快乐开源活动issue与PR
PFCC
Paddle Framework Contributor Club,https://github.com/PaddlePaddle/community/tree/master/pfcc
status/close
已关闭
Comments
paddle-bot
bot
added
the
PFCC
Paddle Framework Contributor Club,https://github.com/PaddlePaddle/community/tree/master/pfcc
label
Jan 10, 2024
This was referenced Jan 23, 2024
This was referenced Jan 24, 2024
Open
28 tasks
Closed
This was referenced Mar 17, 2024
28 tasks
28 tasks
【快乐开源】PIR test_errors 相关单测适配 已全部完成,感谢参与的小伙伴们!
|
github-project-automation
bot
moved this from In Progress
to Done
in Call for Contributions
Apr 18, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
HappyOpenSource
快乐开源活动issue与PR
PFCC
Paddle Framework Contributor Club,https://github.com/PaddlePaddle/community/tree/master/pfcc
status/close
已关闭
一,背景和任务列表
在之前 API 适配 PIR 的时候有⼀些
test_errors
的单测,在 PIR 下并没有适配,因为当时 PIR 下没有
check dtype
相关的逻辑。现在我们已经完成了
check dtype
下沉到 cpp API 的⼯作,所以需要将
test_errors
的测试也适配⼀下 PIR。⭐️ 提交PR 模版 ⭐️:
本期需要升级的 API 如下:
二,任务详情
需要对相关单测进行修改合并,在单测函数中,将
with program_guard(Program(), Program())
与
with paddle.pir_utils.IrGuard(), program_guard(Program(), Program())
合并,并添加test_with_pir_api
装饰器。修改为
合并针对pir和静态图的两个相同用例
修改为
三、参考PR
#60863
#60693
#60695
看板信息
The text was updated successfully, but these errors were encountered: