jump-dev / JuMP.jl

Modeling language for Mathematical Optimization (linear, mixed-integer, conic, semidefinite, nonlinear)
http://jump.dev/JuMP.jl/
Other
2.23k stars 394 forks source link

Documentation suggestions from Discourse #2487

Closed odow closed 3 years ago

odow commented 3 years ago

I put out a call for suggestions on Discourse: https://discourse.julialang.org/t/please-suggest-improvements-to-the-jump-documentation/55596

odow commented 3 years ago

@baggepinnen I'm looking into your suggestion.

What part of the error message isn't clear? https://github.com/jump-dev/JuMP.jl/blob/64b290a85e4e27a2e6f40174d4e9120657f3e3c2/src/sets.jl#L80-L84 Do you have examples of where this error message wasn't sufficient?

baggepinnen commented 3 years ago

If I want to minimize the norm of an expression, I need to rewrite my problem into minimize t and then add the constraint in the error message. That is not obvious to everyone.

odow commented 3 years ago

Closing since these are done for now. I will re-open if there are new suggestions.

sean-lo commented 2 years ago

Hi! I have a comment on the ambiguity in the docstring for RotatedSecondOrderCone() on whether we are taking the 2-norm or squared 2-norm of the vector. The docstring describes it as "the square of the euclidean norm of a vector" while the mathematical formula in the example shows ∥(x−1,x−2)∥_2 ≤ 2tx.