Skip to content

set disable_opt_rtn to optional bool and change default value to None #1231

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.

Sign up for GitHub

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

Jump to bottom
Merged
wenhuach21 merged 5 commits into from
Jan 8, 2026
Merged

set disable_opt_rtn to optional bool and change default value to None #1231

wenhuach21 merged 5 commits into from
Jan 8, 2026

Conversation

@WeiweiZhang1
Copy link
Contributor

@WeiweiZhang1 WeiweiZhang1 commented Jan 7, 2026 •
edited
Loading

No description provided.

@WeiweiZhang1
set disable_opt_rtn to optional bool and change default value from Fa…
0f283f7 
…lse to None

Signed-off-by: Zhang, Weiwei1 <weiwei1.zhang@intel.com>
@WeiweiZhang1 WeiweiZhang1 changed the title set disable_opt_rtn to optional bool and change default value from Fa… set disable_opt_rtn to optional bool and change default value to None Jan 7, 2026
wenhuach21
wenhuach21 reviewed Jan 7, 2026
View reviewed changes
wenhuach21
wenhuach21 reviewed Jan 7, 2026
View reviewed changes
@WeiweiZhang1
fix typo
2b8bc0a 
Signed-off-by: Zhang, Weiwei1 <weiwei1.zhang@intel.com>
wenhuach21
wenhuach21 reviewed Jan 7, 2026
View reviewed changes
wenhuach21
wenhuach21 reviewed Jan 7, 2026
View reviewed changes
WeiweiZhang1 and others added 2 commits January 8, 2026 00:43
@WeiweiZhang1
refine code
c712b2c 
Signed-off-by: Zhang, Weiwei1 <weiwei1.zhang@intel.com>
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
78dda1e 
for more information, see https://pre-commit.ci
@wenhuach21 wenhuach21 self-requested a review January 8, 2026 05:53
@wenhuach21 wenhuach21 requested review from n1ck-guo and yiliu30 January 8, 2026 05:54
wenhuach21
wenhuach21 reviewed Jan 8, 2026
View reviewed changes
wenhuach21
wenhuach21 reviewed Jan 8, 2026
View reviewed changes
wenhuach21
wenhuach21 reviewed Jan 8, 2026
View reviewed changes
wenhuach21
wenhuach21 reviewed Jan 8, 2026
View reviewed changes
yiliu30
yiliu30 reviewed Jan 8, 2026
View reviewed changes
yiliu30
yiliu30 reviewed Jan 8, 2026
View reviewed changes
@wenhuach21 wenhuach21 merged commit bef28c9 into main Jan 8, 2026
28 checks passed
@wenhuach21 wenhuach21 deleted the set_disable_opt_rtn_default_2_none branch January 8, 2026 09:16
@WeiweiZhang1 WeiweiZhang1 mentioned this pull request Jan 8, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yiliu30 yiliu30 yiliu30 left review comments

@wenhuach21 wenhuach21 wenhuach21 approved these changes

@n1ck-guo n1ck-guo Awaiting requested review from n1ck-guo

Assignees

No one assigned

Labels

api/change

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@WeiweiZhang1 @yiliu30 @wenhuach21