-
Notifications
You must be signed in to change notification settings - Fork 5.8k
[CodeStyle][ruff] clean some F401 step: 3 #58306
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提交成功,感谢你对开源项目的贡献! |
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.
2023-10-22 23:28:58 from paddle.distributed.parallel import (
2023-10-22 23:28:58 File "/usr/local/lib/python3.10/dist-packages/paddle/distributed/parallel.py", line 28, in <module>
2023-10-22 23:28:58 from paddle.distributed.collective import (
2023-10-22 23:28:58 File "/usr/local/lib/python3.10/dist-packages/paddle/distributed/collective.py", line 25, in <module>
2023-10-22 23:28:58 from .fleet.layers.mpu.mp_ops import ( # noqa: F401
2023-10-22 23:28:58 File "/usr/local/lib/python3.10/dist-packages/paddle/distributed/fleet/layers/mpu/__init__.py", line 15, in <module>
2023-10-22 23:28:58 from .mp_layers import VocabParallelEmbedding
2023-10-22 23:28:58 File "/usr/local/lib/python3.10/dist-packages/paddle/distributed/fleet/layers/mpu/mp_layers.py", line 21, in <module>
2023-10-22 23:28:58 from paddle.nn import functional as F
2023-10-22 23:28:58 File "/usr/local/lib/python3.10/dist-packages/paddle/nn/__init__.py", line 18, in <module>
2023-10-22 23:28:58 from . import functional, initializer, quant, utils # noqa: F401
2023-10-22 23:28:58 File "/usr/local/lib/python3.10/dist-packages/paddle/nn/quant/__init__.py", line 15, in <module>
2023-10-22 23:28:58 from . import qat # noqa: F401
2023-10-22 23:28:58 File "/usr/local/lib/python3.10/dist-packages/paddle/nn/quant/qat/__init__.py", line 14, in <module>
2023-10-22 23:28:58 from .conv import QuantedConv2D
2023-10-22 23:28:58 File "/usr/local/lib/python3.10/dist-packages/paddle/nn/quant/qat/conv.py", line 20, in <module>
2023-10-22 23:28:58 from ..format import ConvertibleQuantedLayer
2023-10-22 23:28:58 File "/usr/local/lib/python3.10/dist-packages/paddle/nn/quant/format.py", line 21, in <module>
2023-10-22 23:28:58 from paddle.nn import Layer
2023-10-22 23:28:58 ImportError: cannot import name 'Layer' from partially initialized module 'paddle.nn' (most likely due to a circular import) (/usr/local/lib/python3.10/dist-packages/paddle/nn/__init__.py)
circular import 了 [doge]
python/paddle/static/__init__.py
Outdated
@@ -13,71 +13,59 @@ | |||
# See the License for the specific language governing permissions and | |||
# limitations under the License. | |||
|
|||
import os | |||
import os # noqa: F401 |
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.
这个 os……应该没用吧?
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.
冲突需要解决下
from .convert_operators import unpack_by_structure as Unpack # noqa: F401 | ||
|
||
# isort: off | ||
# NOTE(gouzil): isort will delete the import |
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.
TODO:之后添加 combine-as-imports
配置,然后删除这个 isort 的 off 部分
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.
Done
# Conflicts: # python/paddle/nn/__init__.py
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.
LGTMeow 🐾
Sorry to inform you that ebca56a's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually. |
请再解决下冲突 |
# Conflicts: # python/paddle/static/nn/__init__.py
Done |
又冲突了😢 |
# Conflicts: # python/paddle/pir/__init__.py
# Conflicts: # python/paddle/pir/__init__.py
Done |
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.
PR types
Others
PR changes
Others
Description
清理一些
# noqa: F401
存量,整理相关__init__.py
相关链接:
#noqa: F401
step: 1 #57860PGH004
#57941python/paddle/__init__.py
#57682F401
step: 2 #57970