doombeaker
commented
2 years ago
- 作为文档似乎有点口语化了。“当参数不为None时，...；当参数为None时，...”，仅供参考。
- 此处’cuda‘与英文文档一致可以理解，但是作为泛指，device_type是不是会比’cuda‘更好一点。
- 这个pr怎么这么多commit。
第 2条改了。第 1 条保持了原样。 第3 条等下 force push 纠正下。
Open doombeaker opened 2 years ago
doombeaker
commented
2 years ago
- 作为文档似乎有点口语化了。“当参数不为None时，...；当参数为None时，...”，仅供参考。
- 此处’cuda‘与英文文档一致可以理解，但是作为泛指，device_type是不是会比’cuda‘更好一点。
- 这个pr怎么这么多commit。
第 2条改了。第 1 条保持了原样。 第3 条等下 force push 纠正下。
doombeaker
commented
2 years ago https://github.com/Oneflow-Inc/oneflow-api-cn/runs/6700924942?check_suite_focus=true
好像因为 oneflow 仓库代码更新，导致这个 PR 过不了了……
麻烦 @Chenqll 有空时看能不能在本地复现这个问题吧，能复现后，再讨论后面怎么解决。
shangguanshiyuan
commented
2 years ago 我记得这个问题上个月修复了，你把这个文档仓库对应的ci里的oneflow换成较新的oneflow版本试试
原文是：
global_src_rank (int, optional) – The source rank for loading global tensors. When specified, only the process whose rank == global_src_rank will really read the files in path, and tensors in the loaded object will be consistent with placement = flow.placement(‘cuda’, [global_src_rank])