#12220: Updated CBC spkg
--------------------------------------+-------------------------------------
       Reporter:  ncohen              |         Owner:  ncohen    
           Type:  enhancement         |        Status:  needs_work
       Priority:  major               |     Milestone:  sage-5.0  
      Component:  linear programming  |    Resolution:            
       Keywords:  sd35.5, Cernay2012  |   Work issues:            
Report Upstream:  N/A                 |     Reviewers:            
        Authors:  John Perry          |     Merged in:            
   Dependencies:                      |      Stopgaps:            
--------------------------------------+-------------------------------------
Changes (by john_perry):

  * status:  needs_review => needs_work


Comment:

 I know this is a minor thing, but would you mind changing

   * `range(number)` to `xrange(number)` in `coin_backend.pyx`?
   * adding me as an author? I forgot to do that in the patches I
 originally submitted (& some work is mine)

 Otherwise, this is okay, & I would give it a positive review if I could.
 Though I still want to test it on a Mac when I get a chance, since things
 sometimes act funny there.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12220#comment:86>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to