search

ampl / mp

An open-source library for mathematical programming
https://mp.ampl.com
Other
229 stars 42 forks source link

Possibly remove the "data" field #173

Closed mapgccv closed 2 years ago

mapgccv commented 2 years ago

Is the data field of OptionValueInfo being used at all?

https://github.com/ampl/mp/blob/ce2ea2ad039ac0790cb08db8467735bb3a42e260/include/mp/solver-opt.h#L18

glebbelov commented 2 years ago

This is for possible use if a solver needs to store extra infos. Could be useful I think

On Fri, 6 May 2022 at 2:09 am, Christian Valente @.***> wrote:

Is the data field of OptionValueInfo being used at all?

https://github.com/ampl/mp/blob/ce2ea2ad039ac0790cb08db8467735bb3a42e260/include/mp/solver-opt.h#L18

— Reply to this email directly, view it on GitHub https://github.com/ampl/mp/issues/173, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEEBOZTF5UZU4Y725IKOWM3VIPXFBANCNFSM5VFR2Z5A . You are receiving this because you are subscribed to this thread.Message ID: @.***>

mapgccv commented 2 years ago

Uhm, like? I thought i could use the field to map (eventually) an mp solver control value to the solver library one, allowing for example a user to specify a parameter:

method=2

which would internally then passed to setSolverOption as:

setSolverOption("lpmethod", "simplex")

but, AFAICS, the data field is not used anywhere at all.