JuliaML / LIBSVM.jl

LIBSVM bindings for Julia

Other

88 stars 35 forks source link

Fix ScikitLearn API regression example #37

Closed DilumAluthge closed 6 years ago

DilumAluthge commented 6 years ago

Fixes #36.

coveralls commented 6 years ago

Coverage remained the same at 82.383% when pulling 29274f4901f5291dd1152709772e2cd773af5241 on DilumAluthge:patch-1 into 80d48e78a3d643774db41073addf29fddaa14501 on mpastell:master.

codecov-io commented 6 years ago

Codecov Report

Merging #37 into master will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #37   +/-   ##
=======================================
  Coverage   82.38%   82.38%           
=======================================
  Files           4        4           
  Lines         193      193           
=======================================
  Hits          159      159           
  Misses         34       34

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 80d48e7...29274f4. Read the comment docs.

DilumAluthge commented 6 years ago

@mpastell: Can you merge this?

DilumAluthge commented 6 years ago

Bump. @mpastell can you take a look at this?

mpastell commented 6 years ago

Thanks for adding this and sorry for the slow merge!

DilumAluthge commented 6 years ago

No problem, thanks for merging!

On Thu, Jul 12, 2018 at 12:58 Matti Pastell notifications@github.com wrote:

Merged #37 https://github.com/mpastell/LIBSVM.jl/pull/37.

— You are receiving this because you authored the thread.

Reply to this email directly, view it on GitHub https://github.com/mpastell/LIBSVM.jl/pull/37#event-1730815535, or mute the thread https://github.com/notifications/unsubscribe-auth/AFXAraHvxpWP02FIZTEGdfNdKRxqPWtJks5uF4A2gaJpZM4UF7a7 .

--

Dilum Aluthge

dilum@aluthge.com dilum@aluthge.com

https://www.aluthge.com https://www.aluthge.com