JuMP.jl icon indicating copy to clipboard operation
JuMP.jl copied to clipboard

Add DataFrames.jl extension

Open odow opened this issue 1 year ago • 3 comments

Take II of #3441 x-ref #3438

TODOs

  • [x] Add documentation to extension
  • [x] Rework existing tutorials to leverage new syntax

See https://jump.dev/JuMP.jl/previews/PR3821/extensions/DataFrames/

odow avatar Sep 10 '24 02:09 odow

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 97.92%. Comparing base (9fb707c) to head (890ede8). Report is 32 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3821      +/-   ##
==========================================
+ Coverage   97.90%   97.92%   +0.01%     
==========================================
  Files          44       46       +2     
  Lines        6022     6077      +55     
==========================================
+ Hits         5896     5951      +55     
  Misses        126      126              

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Sep 10 '24 02:09 codecov[bot]

Is this an improvement? I'm not sure.

Now: https://jump.dev/JuMP.jl/previews/PR3821/tutorials/linear/multi/ Previous: https://jump.dev/JuMP.jl/stable/tutorials/linear/multi/

odow avatar Sep 10 '24 09:09 odow

We could also have an array with the same syntax as SparseAxisArray and a DataFrame instead of a Dict as a backend. I guess we could also have both the container of your PR and the one I just suggested.

blegat avatar Oct 04 '24 06:10 blegat