-
Notifications
You must be signed in to change notification settings - Fork 5.8k
[3.8] Remove unused dependency astor
#72870
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
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
python/paddle/jit/api.py
Outdated
@@ -156,13 +156,13 @@ def ignore_module(modules: list[ModuleType]) -> None: | |||
.. code-block:: python | |||
|
|||
>>> import scipy | |||
>>> import astor | |||
>>> import torch |
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.
这代码 CI 会跑的,import torch
是什么鬼?改用 decorator
astor
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.
astor
astor
@@ -200,7 +200,6 @@ show_column_numbers = true | |||
|
|||
[[tool.mypy.overrides]] | |||
module = [ | |||
"astor", |
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.
这里加上 decorator
吧,mypy 检查挂了
PR Category
Execute Infrastructure
PR Types
Deprecations
Description
移除第三方库
astor
,其仅用于 AST 动转静,但 AST 动转静很久之前就只在 3.9 及以上版本使用内置的ast.unparse
替代astor
了,而 python 3.8 已经退场,astor
已经彻底无用,故彻底移除Related links