Feedback
未关闭
@goldsborough 已经在做这个了。
开始于 2018年10月8日。
- 主要语言
- HTML
- 星标
- 255
- 派生
- 37
- PR 合并指标
- 30 天内没有已合并 PR
描述
Hey, I just did a simple integration of libtorch as an addon for nodejs. The API worked very nice and I really enjoyed the design, it's very well done ! I'm adding in this issue some feedbacks (don't know if here is the best place, but here you go):
- The
torch::jit::loadisn't exposed by just adding thetorch/torch.hheader, you have to include thetorch/script.has well, don't know if it is intentional but it took me some time to find the right header, maybe a documentation improvement could help; - There are some local paths hardcoded into the ATen cmake files, such as in
share/cmake/ATen/ATenConfig.cmake:/Users/administrator/nightlies/2018_10_01/wheel_build_dirs/libtorch_2.7/pytorch/torch/lib/tmp_install/include; - Documentation is still missing some parts, especially regarding loading TorchScript, etc. Given that a lot of people will start to use the libtorch, I would be nice to have a expanded documentation on ATen/Torch API with some examples (I can contribute as well);
- Problem when seeing the docs for the
torch::jit::load(https://pytorch.org/cppdocs/api/function_namespacetorch_1_1jit_1ace2c44fb8af5905ae17834e81086b8a3.html#exhale-function-namespacetorch-1-1jit-1ace2c44fb8af5905ae17834e81086b8a3) - Many broken broken in the initial page of the docs (
torch::nn,torch::optim,torch::data,torch::serialize,torch::jitandtorch::python)
This is what I remember, if I see anything else I'll add it here as well.
贡献指南
这个仓库没有索引到贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
评估
这个 Issue 还没有评估数据。