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 OpTest Fix No.21】 fix test_data_norm_op #63661

Closed
wants to merge 7 commits into from

Conversation

Eacient
Copy link

@Eacient Eacient commented Apr 18, 2024

PR Category

Others

PR Types

Others

Description

PIR Op单测修复
修复单测 test_data_norm_op
修复后打开FLAGS_enable_pir_in_executor单测是否通过:是

Copy link

paddle-bot bot commented Apr 18, 2024

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot paddle-bot bot added the contributor External developers label Apr 18, 2024
@Eacient Eacient marked this pull request as draft April 18, 2024 09:48
@Eacient Eacient marked this pull request as ready for review April 18, 2024 09:48
@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Apr 19, 2024
@kangguangli kangguangli self-requested a review April 22, 2024 03:10
@luotao1
Copy link
Contributor

luotao1 commented Apr 25, 2024

@Eacient test_data_norm_op (Failed) Coverage流水线单测没有过

Copy link

paddle-ci-bot bot commented Apr 28, 2024

Sorry to inform you that ee0c488's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually.

@luotao1 luotao1 closed this May 11, 2024
@Eacient
Copy link
Author

Eacient commented May 11, 2024

请问后续修改的应该如何提交,需要重开分支吗

@luotao1
Copy link
Contributor

luotao1 commented May 11, 2024

如果要继续修改的话,可以把这个PR reopen,在原来基础上接着修改

@Eacient
Copy link
Author

Eacient commented May 11, 2024

了解,谢谢回复!

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.

3 participants