Page MenuHomeSolus

Update pytorch to 1.1.0
Needs RevisionPublic

Authored by xulongwu4 on May 4 2019, 12:43 PM.

Details

Reviewers
JoshStrobl
Group Reviewers
Triage Team
Summary

Update pytorch to 1.1.0 and add the C++ API.
Changelog is available here.

Test Plan
  • Verified the installation by executing the commands here.
  • Ran the MNIST example successfully.

Diff Detail

Repository
R4672 pytorch
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
xulongwu4 created this revision.May 4 2019, 12:43 PM
xulongwu4 requested review of this revision.May 4 2019, 12:43 PM
JoshStrobl requested changes to this revision.May 9 2019, 5:55 PM
JoshStrobl added a subscriber: JoshStrobl.
JoshStrobl added inline comments.
package.yml
13

What is networking being turned on for?

58–60

Should be able to just do ln -sv $TORCH_PATH/bin/* $installdir/usr/bin/

62–64

See bin comment for likely more optimal method.

65–67

See bin comment for likely more optimal method.

68–70

See bin comment for likely more optimal method.

This revision now requires changes to proceed.May 9 2019, 5:55 PM
xulongwu4 marked an inline comment as done.May 10 2019, 10:38 AM
xulongwu4 added inline comments.
package.yml
13

When running the cmake configure step (invoked by the setup.py script), it will download a package named asmjit. This is what the network is for. One other choice is let the asmjit source code be another source in package.yml and specify its location before running setup.py.

xulongwu4 marked 2 inline comments as done.May 10 2019, 10:53 AM
xulongwu4 added inline comments.
package.yml
58–60

This does not seem a solution in this case. The packages are installed in $installdir/$TORCH_PATH/bin. So $TORCH_PATH/bin/* will just be empty in my opinion.

xulongwu4 marked an inline comment as done.May 13 2019, 4:19 PM
xulongwu4 marked 3 inline comments as done.May 13 2019, 4:22 PM
xulongwu4 added inline comments.
package.yml
62–64

See comments above.

65–67

See comments above.

68–70

See comments above.

xulongwu4 marked 7 inline comments as done.May 13 2019, 4:22 PM
xulongwu4 marked an inline comment as done.

@JoshStrobl Do you have time to take another look at this patch?