-
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
[PIR] Enable program translator in dy2st mode #59152
[PIR] Enable program translator in dy2st mode #59152
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
…program_translator_in_dy2st
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.
@@ -14,6 +14,8 @@ | |||
|
|||
#pragma once | |||
|
|||
#include <iostream> |
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.
这个应该是调试时忘记删除了吧
core._is_fwd_prim_enabled() or core._is_bwd_prim_enabled() | ||
) | ||
in_cinn_backend = self._backend == "CINN" | ||
is_cinn_enabled = self._build_strategy.build_cinn_pass |
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.
is_cinn_enabled = self._build_strategy.build_cinn_pass | |
is_cinn_enabled = self._build_strategy.build_cinn_pass or self._backend == "CINN" |
可以省去一个变量
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.
可以考虑提个commit fix下或者合入后单独提PR
好的,下个PR修复。 |
* enable program translator in dy2st mode * fix * test * test * test * test * fix * fix test_eager_run_program.py * fix test_run_program_op * fix test_gradname_parse partially * fix test_bert_cinn * skip failed cases * skip jit.save * fix test_run_program_op * fix * fix * fix * fix * fix
PR types
Others
PR changes
Others
Description
Enable program translator in dy2st mode, but disable in following cases:
Additional Modifications
in_pir_pt_mode
forrun_porgram
openable_pir_with_pt_in_dy2st
Both of them is used to control whether Pir is enabled in dy2st mode.
Others
Pcard-67164