-
Notifications
You must be signed in to change notification settings - Fork 5.8k
[Typing][C-12] Add type annotations for python/paddle/distributed/communication/all_gather.py
#66051
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提交成功,感谢你对开源项目的贡献! |
补充: 是否可以在 标注 是否可以引入此文件,将需要用到的 |
Update 20240716
尝试去掉 |
在考虑,要不要搞个 p.s. 有个问题是, |
可以呀
如果用户在用开分布式选项的包,那么就一定会有这个 stub,如果用的是没有编分布式的包,那么运行时也不会访问,没有也没什么关系 如果是 CI 检查的话,可以考虑整个 Any 啥的 Fallback 机制吧,感觉也没啥更好的办法 |
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.
…mmunication/all_gather.py` (PaddlePaddle#66051)
PR Category
User Experience
PR Types
Improvements
Description
类型标注:
python/paddle/distributed/communication/all_gather.py
Related links
@SigureMo @megemini