#14666: Test if a weight function is generic for a given matroid
-------------------------------------+-------------------------------------
       Reporter:  Stefan             |        Owner:  Stefanf
           Type:  enhancement        |       Status:  needs_review
       Priority:  minor              |    Milestone:  sage-7.2
      Component:  matroid theory     |   Resolution:
       Keywords:  matroid, weight    |    Merged in:
  function                           |    Reviewers:
        Authors:                     |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:                     |  059b6d3816eae7412bc5e6570c17888fa2b75678
  public/ticket/14666                |     Stopgaps:
   Dependencies:                     |
-------------------------------------+-------------------------------------

Comment (by darij):

 Also, I'm not fully sure about what goes on here:
 {{{
         if weights is None:
             for e in Y:
                 res.append(e)
                 if self._rank(res) > r:
                     r += 1
                 else:
                     del res[-1]
                     smres.append(e)
                     if self._rank(smres) >= 1:
                         return False
                     else:
                         del smres[-1]
             return True
 }}}
 Am I seeing it right that `smres` never has length >1 ? So you are
 essentially checking that `e` is a loop; if it isn't, then you return
 `False`? Do you really need an array for that?

--
Ticket URL: <http://trac.sagemath.org/ticket/14666#comment:15>
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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to