-
Notifications
You must be signed in to change notification settings - Fork 825
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
fix tensor method docstr bug #7841
Conversation
CI failed when running job: cuda-module-distributed-rank-1. PR label automerge has been removed |
CI failed when running job: cuda-module-distributed-rank-0. PR label automerge has been removed |
我发现,我用
oneflow 版本: (oneflow-dev-gcc7-v2) yaochi@oneflow-23:~/build_exp$ python -m oneflow --doctor
path: ['/home/yaochi/oneflow/python/oneflow']
version: 0.8.0+cu112.git.1875c6e2b3
git_commit: 1875c6e2b3
cmake_build_type: Release
rdma: False
mlir: False 有点奇怪,要不要核对下是不是oneflow版本的问题?因为我有点记忆,后江修过类似的问题。我本来以为是还没考虑全情况,不过我这边测又没问题…… |
CI failed when running job: cuda-module-distributed-rank-0. PR label automerge has been removed |
Speed stats:
|
修复了一部分Tensor方法没有docstr的bug。
剩下的:
是通过pybind11直接导出的,直接设置docstr会触发错误,后面需要和迟哥讨论一下再解决。
最后:norm这个算子的docstr比较复杂,设计到pytorch的新老接口替换的问题,也需要再讨论一下。
这个pr可以先合并,以下是本次补充的docstr的截图。