Opened 8 years ago
Closed 8 years ago
#17463 closed defect (fixed)
Remove the deprecated 'dim' argument of MIP variables
Reported by: | ncohen | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | sage-6.7 |
Component: | numerical | Keywords: | |
Cc: | dimpase | Merged in: | |
Authors: | Nathann Cohen | Reviewers: | Dima Pasechnik |
Report Upstream: | N/A | Work issues: | |
Branch: | d3c8f90 (Commits, GitHub, GitLab) | Commit: | d3c8f902e77b87fb1a1f6406d04a04127683f452 |
Dependencies: | Stopgaps: |
Description (last modified by )
The 'dim' attribute of MIPVariables has been deprecated since Sage 6.2 in
#15489. It was always useless since myvar[a,b]
works and is more practical
than myvar[a][b]
. I had not noticed that when I implemented it
T_T
With this ticket, the problem is solved.
Change History (17)
comment:1 Changed 8 years ago by
Branch: | → u/ncohen/17463 |
---|---|
Status: | new → needs_review |
comment:2 Changed 8 years ago by
Commit: | → d3c8f902e77b87fb1a1f6406d04a04127683f452 |
---|
comment:5 Changed 8 years ago by
Description: | modified (diff) |
---|---|
Milestone: | sage-6.5 → sage-pending |
comment:6 Changed 8 years ago by
Summary: | Remove the (deprecate) 'dim' argument of MIP variables → Remove the deprecated 'dim' argument of MIP variables |
---|
comment:8 Changed 8 years ago by
I do not object to changing the milestone and having it merged tomorrow, if you ask.
Nathann
comment:10 Changed 8 years ago by
Reviewers: | → Dima Pasechnik |
---|
comment:11 follow-up: 12 Changed 8 years ago by
PSA: Once you want the ticket merged you can set a milestone to something else than "sage-pending"
comment:12 Changed 8 years ago by
PSA: Once you want the ticket merged you can set a milestone to something else than "sage-pending"
Right right. I asked google calendar to send me an email on the 2015-05-06.
Nathann
comment:13 Changed 8 years ago by
Just trying to make sure that everybody understands that feature of the merge script since it caused some confusion on another ticket
comment:14 Changed 8 years ago by
Milestone: | sage-pending → sage-6.7 |
---|
comment:16 Changed 8 years ago by
Description: | modified (diff) |
---|
comment:17 Changed 8 years ago by
Branch: | u/ncohen/17463 → d3c8f902e77b87fb1a1f6406d04a04127683f452 |
---|---|
Resolution: | → fixed |
Status: | positive_review → closed |
Branch pushed to git repo; I updated commit sha1. New commits:
Removed 'MIPVariable.dim' attribute deprecated in #15489 on 2013-12-06