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

[Dy2St] pir dy2st unittest verification - Part 10 #59276

Merged
merged 9 commits into from
Nov 28, 2023

Conversation

gouzil
Copy link
Member

@gouzil gouzil commented Nov 22, 2023

PR types

Others

PR changes

Others

Description

状态 单测 错误类别 备注 报错信息
test_deepcopy
test_set_dynamic_shape API 需要适配paddle.jit.dy2static.utils_helper.set_dynamic_shape,添加类似 OpResult._set_shape 之类的接口,已反馈震哥 AttributeError: 'paddle.base.libpaddle.pir.OpResult' object has no attribute 'desc'
🚧 test_partial_program: {TestWithNoGrad, TestWithTrainAndEval} 修改单测即可 API 需要适配paddle.jit.dy2static.pir_partial_program.PartialProgramLayer._train_program, 或者修改单测(TestWithNoGrad有个遗留问题) AttributeError: 'PartialProgramLayer' object has no attribute '_train_program'
test_partial_program: TestPruneUnusedParamInProgram 动转静执行 run program op 报错 RuntimeError: (NotFound) Cannot find parameter_0 in scope
🚧 test_loop: {TestTransformWhileLoop, TestTransformForLoop} 控制流 暂不支持控制流 ValueError: (InvalidArgument) Currently, we can only get name of OpResult that is persistable
test_loop: TestForLoopMeetDict API 需要适配jit.api.save方法 AttributeError: 'list' object has no attribute 'type'
test_lac 跳过
test_ptb_lm_v2 待确定 SOT + PIR API 段错误
test_ptb_lm_v2 动转静转写 AST + PIR API 挂在paddle.matmul,本问题 #59224 已修复 ValueError: (InvalidArgument) matmul(): argument (position 2) must be OpResult, but got EagerParamBase
test_ptb_lm_v2 动转静执行 同 part 9 里的 test_mnist,append_backward 时候段错误

相关链接:

Copy link

paddle-bot bot commented Nov 22, 2023

你的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 Nov 22, 2023
@gouzil
Copy link
Member Author

gouzil commented Nov 23, 2023

截屏2023-11-23 上午10 14 14

这个我本地也小概率遇到,需要看看

@gouzil gouzil requested a review from SigureMo November 23, 2023 02:15
SigureMo
SigureMo previously approved these changes Nov 23, 2023
Copy link
Member

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

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

LGTMeow

Copy link
Member

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

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

LGTMeow

@gouzil 这个 PR 可以同步到表格了

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants