JuliaML / MLPreprocessing.jl

Other
15 stars 2 forks source link

WIP: Add more tests #2

Closed abieler closed 7 years ago

abieler commented 7 years ago

Just adding tests to get to ~100% coverage.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+11.7%) to 79.464% when pulling 2e47b6262a83168810b7b091950cb0dceb25861b on more_tests into 487e60177701d54c27a27c02eb3bdb0ebc626461 on master.

Evizero commented 7 years ago

nice

Evizero commented 7 years ago

Oh, was I prematurely merging? If so sorry, the tests passed and the code looked uncontroversial so I may have just jumped the gun

abieler commented 7 years ago

Yes, I am still adding more tests :) Had me confused for a second though.

Evizero commented 7 years ago

Yea the WIP in the title should have given me a strong hint, but my brain is somewhere else today