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 11 #59314

Merged
merged 5 commits into from
Nov 29, 2023

Conversation

gouzil
Copy link
Member

@gouzil gouzil commented Nov 23, 2023

PR types

Others

PR changes

Others

Description

状态 单测 错误类别 备注 报错信息
test_simnet_v2 API test_simnet修复了再看看,报错是一样的(依赖 @zhangbo9674) #59312 已修复
test_simnet API #59312 已修复
test_build_strategy 反向 InvalidArgumentError: The type of data we are trying to retrieve (uint8) does not match the type of data (float32) currently contained in the container.
test_sentiment 动转静执行 段错误, 挂在forward里面了
test_load_transformer
test_save_load API paddle.jit.api.save需要适配 AttributeError: 'list' object has no attribute 'type'

相关链接:

Copy link

paddle-bot bot commented Nov 23, 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.

@SigureMo
Copy link
Member

TODO: #59364 应该会修复 simnet,等合入后复测下

@SigureMo
Copy link
Member

TODO: test_sentiment PT 就有精度问题,该单测不知道什么时候被禁用了,现在单测机制关不掉 PT,需等待 #59312 合入后测一下老 IR 是否有同样的问题

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

test_sentiment 很久之前就禁用了

https://github.com/PaddlePaddle/PaddleSOT/blob/develop/tests/run_all_paddle_ci.sh#L11

这个单测先低优下,优先看其他单测和最近才被禁用的单测

这个 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.

3 participants