search

JuliaML / MLDataUtils.jl

Utility package for generating, loading, splitting, and processing Machine Learning datasets
http://mldatautilsjl.readthedocs.io/
Other
102 stars 20 forks source link

Fix FeatureNormalizer to handle Integer Matrix #22

Closed asbisen closed 7 years ago

asbisen commented 7 years ago

This fix allows Integer Matrix to be passed to predict() for FeatureNormalizer

Evizero commented 7 years ago

Thanks. You didn't have to create a new PR if you add another commit. PRs are branch specific so they update if you push changes to that branch (in your case the master branch of your fork).

Evizero commented 7 years ago

Maybe as a last git exercise, can you squash the two commits into one? Since the second one undoes the first one, the first one serves no purpose

Evizero commented 7 years ago

Nice. Thanks!