Opened 7 years ago

Closed 7 years ago

#15489 closed defect (fixed)

deprecate the dim argument for MIPVariables

Reported by: ncohen Owned by:
Priority: major Milestone: sage-6.2
Component: linear programming Keywords:
Cc: ppurka, kcrisman, dimpase Merged in:
Authors: Nathann Cohen Reviewers: Dima Pasechnik
Report Upstream: N/A Work issues:
Branch: cb4e41b (Commits, GitHub, GitLab) Commit: cb4e41b02160c54a3a3a4a429f0cfd8f49cefc54
Dependencies: #15482 Stopgaps:

Status badges

Description

I implemented this "dim" argument so that my_variable[2][3] would work, and before noticing that my_variable[2,3] worked too, and actually better. Thus this useless code should be removed, and I was probably the only one to use it in the first place :-)

Nathann

Change History (15)

comment:1 Changed 7 years ago by ncohen

  • Branch set to u/ncohen/15489
  • Cc dimpase added
  • Status changed from new to needs_review

Wow. I did not realise there were so many LP in Sage O_o

Nathann

comment:2 Changed 7 years ago by git

  • Commit set to 2156b3a9c0d84f38d92820925f88cb7647288b6c

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

​2156b3atrac #15489: deprecate the dim argument for MIPVariables
​9a66631trac #15482: Scream where I had not thought of screaming before
​b50d7aftrac #15482: addressing the reviewer's comments
​1f6c156trac #15482: Say very loud that LP variables are positive by default

comment:3 Changed 7 years ago by git

  • Commit changed from 2156b3a9c0d84f38d92820925f88cb7647288b6c to 8a901ae4508000a72c535cc91adb6001d64954bd

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

​8a901aetrac #15489: rebase on 5.13.rc0

comment:4 Changed 7 years ago by git

  • Commit changed from 8a901ae4508000a72c535cc91adb6001d64954bd to eeb4e8c2e0bd5724682fb50c10fb37300918c470

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

​eeb4e8ctrac #15489: fix doctests and adds notes to remove code later

comment:5 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:6 Changed 7 years ago by git

  • Commit changed from eeb4e8c2e0bd5724682fb50c10fb37300918c470 to 60806600947f8d9cecb0e9805a81e1ba7484f22c

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

​d2870e1trac #15482: shortening a line
​cd5bb43trac #15482: positive -> non-negative
​dd7bde8trac #15482: a broken doctest
​c8addd9trac #15482: rebase on 5.13.rc0
​076f85btrac #15482: Rebase on 6.2.beta0
​b87f1detrac #15482: Rebase on 6.2.beta1
​9a7657btrac #15482: Broken doctest
​6080660trac #15489: Rebase on updated #15482

comment:7 Changed 7 years ago by dimpase

I'm reviewing this one now.

comment:8 Changed 7 years ago by dimpase

I am getting 

[numerical] docstring of sage.numerical.mip.MixedIntegerLinearProgram.set_binary:27: WARNING: Literal block expected; none found.

and the docbuild bails out with OSError at the same line. Not sure which line of mip.pyx is causing the trouble...

comment:9 Changed 7 years ago by dimpase

  • Status changed from needs_review to needs_work

comment:10 Changed 7 years ago by git

  • Commit changed from 60806600947f8d9cecb0e9805a81e1ba7484f22c to cb4e41b02160c54a3a3a4a429f0cfd8f49cefc54

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

​33d5c9ftrac #15489: Rebase on 6.2.beta2
​cb4e41btrac #15489: Docfix

comment:11 Changed 7 years ago by ncohen

  • Status changed from needs_work to needs_review

comment:12 Changed 7 years ago by dimpase

  • Status changed from needs_review to positive_review

looks good.

comment:13 Changed 7 years ago by ncohen

Thaaaaaaaaaaaaaanks !!!

Nathann

comment:14 Changed 7 years ago by vbraun

  • Reviewers set to Dima Pasechnik

comment:15 Changed 7 years ago by vbraun

  • Branch changed from u/ncohen/15489 to cb4e41b02160c54a3a3a4a429f0cfd8f49cefc54
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.