Design Suggestion: Oracle Helper Fcns about jumper.jl HOT 4 CLOSED

This is very on my mind, and indeed #2 was me capturing that though. So we are definitely on the same page. I actually think its pretty exciting how the polyhedral specific part of PolyhedralOracle (apart from the setup) is realllly short.

from jumper.jl.

Ok, so that's what that meant. :) The original issue was a bit cryptic. :)

A related worry that sparked this line of thought was that although
defining UncertainConstr as GenericRangeConstr and type aliasing is
elegant, it leads to potentially confusing code like accessing UAffExpr via

unc_constraint.coeffs[1]

(in my mind, coeffs are floats, but maybe i'm just old-fashioned.)

For people new to JuMPer, this is definitely a weird gotchya. Not
something that a researcher wants to have to decipher in order to code up a
custom oracle. Possibly unrelated. to this thread.

Vishal Gupta
PhD Candidate, 2014
Operations Research Center
Massachusetts Institute of Technology
www.mit.edu/~vgupta1/

On Sun, Apr 20, 2014 at 11:33 PM, Iain Dunning [email protected]:

This is very on my mind, and indeed #2https://github.com/IainNZ/JuMPeR.jl/issues/2was me capturing that though. So we are definitely on the same page. I
actually think its pretty exciting how the polyhedral specific part of
PolyhedralOracle (apart from the setup) is realllly short.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/26#issuecomment-40913942
.

from jumper.jl.

Yeah so in the beginning:

UAffExpr - coeffs, uncs
FullAffExpr - coeffs::Vector{UAffExpr}, vars

then I made FullAffExpr make use of GenericAffExpr which cut down on code (and thus chances for bugs).

Finally, I changed recently the UAffExpr to also be a GenericAffExpr to cut down code any more - but at the cost of losing uncs and changing it to vars. I'm happy with coeffs but vars kind of hurts. I'm hoping that the helper functions will eventually wrap this to the point where even oracle developers don't need to touch JuMPeR-specific stuff like this.

from jumper.jl.

Fixed to my satisfaction for now, need to let whats there sit and see how it feels

from jumper.jl.