Skip to content

Commit

Permalink
add test for #124
Browse files Browse the repository at this point in the history
  • Loading branch information
joehuchette committed Mar 7, 2014
1 parent 0633cc8 commit 7ce5bba
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions test/variable.jl
Original file line number Diff line number Diff line change
Expand Up @@ -95,3 +95,6 @@ idx3 = [[2:2:20],21]
@defVar(mprint, q[idx2,idx3])
@test JuMP.dictstring(q, :REPL) == "q[i,j], for all i in {2,4..18,20}, j in {2,4,6,8,10,12..} free"
@test JuMP.dictstring(q, :IJulia) == "q_{i,j} \\quad \\forall i \\in \\{ 2,4..18,20 \\}, j \\in \\{ 2,4,6,8,10,12.. \\} free"
# test empty JuMPDict printing (issue #124)
@defVar(mprint, xx[1:0])
@test JuMP.dictstring(xx, :REPL) == JuMP.dictstring(xx, :IJulia) == ""

5 comments on commit 7ce5bba

@mlubin
Copy link
Member

@mlubin mlubin commented on 7ce5bba Mar 7, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool. Give me a chance to go through everything, tweak the docs, and I'll tag 0.4 this weekend.

@joehuchette
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@IainNZ
Copy link
Collaborator

@IainNZ IainNZ commented on 7ce5bba Mar 7, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about #118 and maybe #122 ?

@mlubin
Copy link
Member

@mlubin mlubin commented on 7ce5bba Mar 8, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@joehuchette I've tweaked the docs relating to internal models. Let me know if it sounds okay now.

@joehuchette
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Please sign in to comment.