JuliaML / MLUtils.jl

Utilities and abstractions for Machine Learning tasks
MIT License
107 stars 20 forks source link

use buffer in BatchView #157

Open CarloLucibello opened 1 year ago

CarloLucibello commented 1 year ago

fix #156

This covers only BatchView(...; collate=nothing), I'm not sure how collate=true and collate=false should interact with the buffer so I won't address that here.

codecov-commenter commented 1 year ago

Codecov Report

Merging #157 (661f09e) into main (2677a73) will increase coverage by 0.11%. The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##             main     #157      +/-   ##
==========================================
+ Coverage   88.46%   88.57%   +0.11%     
==========================================
  Files          13       15       +2     
  Lines         598      604       +6     
==========================================
+ Hits          529      535       +6     
  Misses         69       69              
Impacted Files Coverage Δ
src/batchview.jl 82.35% <100.00%> (+1.50%) :arrow_up:

... and 2 files with indirect coverage changes