Skip to content

Conversation

@DrRyanHuang
Copy link
Contributor

@DrRyanHuang DrRyanHuang commented Apr 25, 2025

PR Category

Execute Infrastructure

PR Types

New features

Description

添加 SotCapturedException 类,在SOT模拟字节码执行过程中,将所遇到的 Exception 映射为 SotCapturedException,方便SOT对Python异常处理相关字节码的模拟

PCard-66972

@paddle-bot
Copy link

paddle-bot bot commented Apr 25, 2025

你的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.

@paddle-bot paddle-bot bot added the contributor External developers label Apr 25, 2025
@DrRyanHuang
Copy link
Contributor Author

@SigureMo Please review🫶

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 🐾

不影响合入

Comment on lines +404 to +410
origin_exc: Exception | None = None,
exc_type: type[Exception] | None = None,
args: list[Any] | tuple[Any] | None = None,
context: Exception | None = None,
cause: Exception | None = None,
suppress_context: bool | None = None,
traceback: None = None,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里需要这么多可选值吗?没必要的不建议用可选值,args 有必要用三种类型吗?建议用一种,tuple 类型标注也不对,如果是多个元素应该是 tuple[Any, ...]

@SigureMo SigureMo changed the title [SOT][3.13] Add SotCapturedException [SOT] Add SotCapturedException Apr 28, 2025
@SigureMo SigureMo merged commit 63bf3a7 into PaddlePaddle:develop Apr 28, 2025
41 of 42 checks passed
@DrRyanHuang DrRyanHuang deleted the add_sot_captured_exception branch April 29, 2025 02:19
YqGe585 pushed a commit to YqGe585/Paddle that referenced this pull request May 7, 2025
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.

2 participants