beacon-biosignals / LegolasFlux.jl

Save Flux model weights in Legolas-powered Arrow tables
MIT License
6 stars 1 forks source link

make `(read|write)_model_row` support custom schemas #26

Closed kleinschmidt closed 1 year ago

kleinschmidt commented 1 year ago

Fixes #25 by allowing read/write model row to handle custom schemas:

I've added tests and updated the digits example, confirming that it can run end-to-end and produces the same results as the original. I re-committed the saved arrow file in order to be able to use the embedded schema info via read_model_row

codecov[bot] commented 1 year ago

Codecov Report

Merging #26 (e2e7b5c) into main (53c6778) will increase coverage by 0.24%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #26      +/-   ##
==========================================
+ Coverage   93.87%   94.11%   +0.24%     
==========================================
  Files           2        2              
  Lines          49       51       +2     
==========================================
+ Hits           46       48       +2     
  Misses          3        3              
Impacted Files Coverage Δ
src/LegolasFlux.jl 91.66% <100.00%> (+0.49%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more