Opened 6 years ago

Last modified 6 years ago

#17463 closed defect

Remove the (deprecate) 'dim' argument of MIP variables — at Version 5

Reported by: ncohen Owned by:
Priority: major Milestone: sage-6.7
Component: numerical Keywords:
Cc: dimpase Merged in:
Authors: Nathann Cohen Reviewers:
Report Upstream: N/A Work issues:
Branch: u/ncohen/17463 (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.

Not to be merged before 2015-05-06

Change History (5)

comment:1 Changed 6 years ago by ncohen

  • Branch set to u/ncohen/17463
  • Status changed from new to needs_review

comment:2 Changed 6 years ago by git

  • Commit set to 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 6 years ago by dimpase

  • Status changed from needs_review to positive_review

LGTM

comment:4 Changed 6 years ago by ncohen

Thanks !

Nathann

comment:5 Changed 6 years ago by jdemeyer

  • Description modified (diff)
  • Milestone changed from sage-6.5 to sage-pending
Note: See TracTickets for help on using tickets.