search

JuliaML / MLUtils.jl

Utilities and abstractions for Machine Learning tasks
MIT License
109 stars 22 forks source link

add shuffle to splitobs #43

Closed CarloLucibello closed 2 years ago

CarloLucibello commented 2 years ago

unfortunately we loose inferrability in some cases, but I think it is convenient to have this and inferrability may improve in future julia versions.

I also reduced the verbosity of some docstrings.

codecov-commenter commented 2 years ago

Codecov Report

Merging #43 (60ae845) into main (b4e57a2) will increase coverage by 0.02%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #43      +/-   ##
==========================================
+ Coverage   88.91%   88.94%   +0.02%     
==========================================
  Files          13       13              
  Lines         379      380       +1     
==========================================
+ Hits          337      338       +1     
  Misses         42       42
Impacted Files Coverage Δ
src/splitobs.jl 100.00% <100.00%> (ø)

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 b4e57a2...60ae845. Read the comment docs.

CarloLucibello commented 2 years ago

All @inferred tests pass on julia 1.6, a few of them fail on 1.7 and most of them fail on nightly. Oddly julia is getting worse