Don’t use internal JuMP fields

[ericphanson]

TravelingSalesmanExact.jl/src/TravelingSalesmanExact.jl

Line 247 in 5b1a9fc

@info "Final problem has $(length(model.variable_to_zero_one)) binary variables, $(num_constraints(model, GenericAffExpr{Float64,VariableRef}, MOI.LessThan{Float64})) inequality constraints, and $(num_constraints(model, GenericAffExpr{Float64,VariableRef}, MOI.EqualTo{Float64})) equality constraints."

This causes exceptions on Travis (https://travis-ci.com/ericphanson/TravelingSalesmanExact.jl), although on Julia 1.0 at least, they don’t cause test failures for some reason. I should check back once the rest of the tests complete.

I should add compat to JuMP too.

[ericphanson]

I don’t think this is why PkgEval failed though (JuliaLang/julia#32973 (comment)). It looks like tests timeout on 1.2+. This might be due to type inference regressions, since MOI had similar problems on 1.2.

Edit: as noted here JuliaLang/julia#32973 (comment)

[ericphanson]

I made a separate issue to track the inference problems: #7.