search

FluxML / Torch.jl

Sensible extensions for exposing torch in Julia.
Other
213 stars 15 forks source link

Clarify if Python is needed in the README #39

Closed juliohm closed 4 years ago

juliohm commented 4 years ago

Can you please clarify if Python is needed? Do you interface with Torch directly or Python must be installed?

DhairyaLGandhi commented 4 years ago

There is no python dependency with this package. We use the cpp binaries and expose the same kernels as torch, without needing python in the mix.

juliohm commented 4 years ago

Perhaps making it explicit in the README would be a good idea. You mention PyTorch there and so people may think Python is playing a role somewhere:

This package is aimed at providing the Tensor type, which offloads all computations over to PyTorch.

DhairyaLGandhi commented 4 years ago

A PR with the preferred wording would be appreciated!

DhairyaLGandhi commented 4 years ago

Closed on master