[Dy2St] Drop legacy IR support in SOT mode#71520
[Dy2St] Drop legacy IR support in SOT mode#71520SigureMo merged 8 commits intoPaddlePaddle:developfrom
Conversation
|
你的PR提交成功,感谢你对开源项目的贡献! |
|
Sorry to inform you that 360e176's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually. |
There was a problem hiding this comment.
Pull Request Overview
Drop legacy IR support in SOT mode by enforcing PIR mode in the symbolic translator, preventing misconfigured runs.
- Added a runtime check at the start of
symbolic_translateto ensure PIR mode is enabled and raise an error otherwise.
Comments suppressed due to low confidence (1)
python/paddle/jit/sot/translate.py:91
- Add a unit test to verify that
symbolic_translateraises aRuntimeErrorwhenuse_pir_api()returns false, ensuring this guard is covered.
if not paddle.framework.use_pir_api():
Co-authored-by: Copilot <[email protected]>
Codecov ReportAttention: Patch coverage is
❌ Your patch status has failed because the patch coverage (50.00%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## develop #71520 +/- ##
==========================================
Coverage ? 50.00%
==========================================
Files ? 1
Lines ? 2
Branches ? 0
==========================================
Hits ? 1
Misses ? 1
Partials ? 0 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
DrRyanHuang
left a comment
There was a problem hiding this comment.
PIR不是已经切换成默认模式了吗?哪些case还会用到老IR呀
训练的话,一般来说没有场景,导出的话,有些用户仍然希望能够导出 |
PR Category
Execute Infrastructure
PR Types
Bug fixes
Description
由于最近迭代的原因,目前 SOT+老 IR 已经跑不起来了,但是报错并不友好,因此彻底在 SOT 下禁掉老 IR,避免经常因为跑错模式而出现奇怪的报错
之所以只禁地掉 SOT 下的老 IR 是因为 SOT 不面向导出,只面向训练,而 AST 目前主要面向导出,部分场景下仍然需要,因此还不能禁掉