-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
[Typing][C-4][C-7][C-9][C-18]Add type annotations for python/paddle/distributed/auto_parallel/interface.py+strategy.py+placement_type.py and python/paddle/distributed/communication/group.py #66710
Conversation
…_parallel/ interface.py
你的PR提交成功,感谢你对开源项目的贡献! |
…_parallel/ interface.py
…_parallel/ interface.py
…_parallel/ interface.py
…distributed/auto_parallel/interface.py+strategy.py+placement_type.py and python/paddle/distributed/communication/group.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.
不是 C-4
么?怎么改了这么多文件?先从一个文件开始吧……
另外请注意提示信息 ⚠️提示⚠️ python/paddle/distributed 模块难度稍高,可先认领其他模块任务 🤟🤟🤟
,先不要做分布式的……不然只会浪费彼此时间……
@lwkhahaha C-4,C-7 先不用标了,可以恢复掉。 图中的错误是类型标注有问题,需要检查一下。 另外,PR 模板注意一下,不然 Tracking Issue 跟踪不到 |
Sorry to inform you that 1ab8a5d's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually. |
PR Category
Others
PR Types
Others
Description
Add type annotations for python/paddle/distributed/auto_parallel/ interface.py