-
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
[Dy2St][PIR] Enable PIR ut test_sentiment
and remove test_lac
#63192
[Dy2St][PIR] Enable PIR ut test_sentiment
and remove test_lac
#63192
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
❌ The PR is not created using PR's template. You can refer to this Demo. |
test_sentiment
test_sentiment
test_sentiment
and remove test_lac
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 Category
Execute Infrastructure
PR Types
Others
Description
开启单测
test_sentiment
,基于 #63190,SOT 下 PT 和老 IR 有精度问题,暂不排查移除已经禁用好久了的单测
test_lac
(至少当初推 SOT 的时候就禁用掉了),该单测包含 3 个直接裸调_C_ops
的 API(linear_chain_crf
、crf_decoding
、chunk_eval
),根据 3.0 规范,这 3 个 API 均为待删除 API,因此该单测没必要在 PIR 下适配Pcard-67164