add MOI to LP conversion #32
Conversation
Codecov ReportBase: 87.25% // Head: 88.94% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #32 +/- ##
==========================================
+ Coverage 87.25% 88.94% +1.68%
==========================================
Files 5 6 +1
Lines 361 416 +55
==========================================
+ Hits 315 370 +55
Misses 46 46
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
|
related to jump-dev/MathOptInterface.jl#1940 @odow, we should only support MOI to a single LP form. The other conversions should follow from that point. @blegat, currently the conic geometric form is very flexible here. a change to MatrixOfConstraints would limit the available sets to a fixed subset, right? Have you thought about that? |
|
I think MatrixOptInterface should do the matrix conversion to MOI both ways. after this, MatOI will be able to do it will be missing: |
| for (src_ci, cache_ci) in src_to_cache.con_map | ||
| index_map[src_ci] = cache_to_dest[cache_ci] | ||
| end | ||
| return index_map |
There was a problem hiding this comment.
Can't you just return src_to_cache ?
We can move https://github.com/jump-dev/DiffOpt.jl/blob/master/src/product_of_sets.jl to this repo and use it for the conic version. |
No description provided.