search

cstjean / ScikitLearn.jl

Julia implementation of the scikit-learn API https://cstjean.github.io/ScikitLearn.jl/dev/
Other
546 stars 75 forks source link

TODO: Fix remaining Python dependencies #31

Open cstjean opened 7 years ago

cstjean commented 7 years ago

train_test_split and all the code relying on importpy (as well as that function itself) ought to be reviewed and fixed to avoid loading scikit-learn unnecessarily.

ValdarT commented 5 years ago

Hey Any chance you'll revisit this issue soon? In general, I think it would be a huge improvement if all the Python dependencies were optional - it's rather impractical to use this library in any inference code/service otherwise.

cstjean commented 5 years ago

Realistically, that seems unlikely in the near future, but PRs are very welcome.