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

  * status:  needs_work => needs_review
  * reviewer:  => John Perry, Nathann Cohen
  * author:  John Perry => John Perry, Nathann Cohen


Comment:

 Hellooooooo !!!

 >   * `range(number)` to `xrange(number)` in `coin_backend.pyx`?

 Hmmmm... You think it is faster ? I thought that the "for i in range(n)"
 were directly translated to C loops when used in a Python file.

 Actually, you can do "sage -cython -a file.pyx" to see that. It creates a
 html file from the pyx file, and you can see by double-clicking on a line
 how it is translated in C `:-)`

   * adding me as an author? I forgot to do that in the patches I
 originally submitted (& some work is mine)

 Oh sorry ! You mean in the HG patches ? Actually the "all tests pass"
 contains only your name as the author (I folded many patches and the first
 one was probably yours) `:-)`

 > 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.

 No problem ! I am glad to this this nightmarish ticket (almost) settled
 `:-)`

 Nathann

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12220#comment:87>
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