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:

Status badges

Description (last modified by jdemeyer)

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 ncohen

Branch: u/ncohen/17463
Status: newneeds_review

comment:2 Changed 8 years ago by git

Commit: d3c8f902e77b87fb1a1f6406d04a04127683f452

Branch pushed to git repo; I updated commit sha1. New commits:

d3c8f90Removed 'MIPVariable.dim' attribute deprecated in #15489 on 2013-12-06

comment:3 Changed 8 years ago by dimpase

Status: needs_reviewpositive_review

LGTM

comment:4 Changed 8 years ago by ncohen

Thanks !

Nathann

comment:5 Changed 8 years ago by jdemeyer

Description: modified (diff)
Milestone: sage-6.5sage-pending

comment:6 Changed 8 years ago by jdemeyer

Summary: Remove the (deprecate) 'dim' argument of MIP variablesRemove the deprecated 'dim' argument of MIP variables

comment:7 Changed 8 years ago by vbraun

I hope you don't expect me to merge "sage-pending" tickets...

comment:8 Changed 8 years ago by ncohen

I do not object to changing the milestone and having it merged tomorrow, if you ask.

Nathann

comment:9 Changed 8 years ago by vbraun

I'm neither the author nor the reviewer (name missing btw).

comment:10 Changed 8 years ago by ncohen

Reviewers: Dima Pasechnik

comment:11 Changed 8 years ago by vbraun

PSA: Once you want the ticket merged you can set a milestone to something else than "sage-pending"

comment:12 in reply to:  11 Changed 8 years ago by ncohen

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 vbraun

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 jdemeyer

Milestone: sage-pendingsage-6.7

comment:15 Changed 8 years ago by ncohen

Oh. Thanks :-D

comment:16 Changed 8 years ago by jdemeyer

Description: modified (diff)

comment:17 Changed 8 years ago by vbraun

Branch: u/ncohen/17463d3c8f902e77b87fb1a1f6406d04a04127683f452
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.