Skip to content

Conversation

@SigureMo
Copy link
Member

@SigureMo SigureMo commented Dec 25, 2023

PR types

Others

PR changes

Others

Description

#60246 遗留问题,清理剩余的 astor 使用,统一使用 ast_to_source_code

PCard-66972

@paddle-bot
Copy link

paddle-bot bot commented Dec 25, 2023

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

Copy link
Member

@gouzil gouzil left a comment

Choose a reason for hiding this comment

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

LGTM

@gouzil
Copy link
Member

gouzil commented Dec 25, 2023

需不需要在ci拦截一下

@SigureMo
Copy link
Member Author

需不需要在ci拦截一下

只有动转静转写逻辑会用,动转静 transformer 已经趋于稳定,基本不会添加新逻辑了

@SigureMo SigureMo merged commit bae3687 into PaddlePaddle:develop Dec 28, 2023
@SigureMo SigureMo deleted the dy2st/replace-all-astor-usage-with-ast-to-source-code branch December 28, 2023 06:10
Wanglongzhi2001 pushed a commit to Wanglongzhi2001/Paddle that referenced this pull request Jan 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants