#15295: LinearMatroid.has_line_minor should catch NotImplementedError
-------------------------------------+-------------------------------------
       Reporter:  Stefan             |        Owner:
           Type:  defect             |       Status:  needs_info
       Priority:  minor              |    Milestone:  sage-6.2
      Component:  matroid theory     |   Resolution:
       Keywords:                     |    Merged in:
        Authors:                     |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/Rudi/ticket/15295                |  dd3ad7d2943d03db3ad8e1001362704ece1ed77b
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by Rudi):

 * status:  new => needs_info
 * commit:   => dd3ad7d2943d03db3ad8e1001362704ece1ed77b


Comment:

 Added that exception clause. Now the same example raises another
 NotImplementedError, but one that gives more information. Essentially,
 computing ratios in that given ring Q is not possible, which is a problem
 in LinearMatroid._line_ratios().

 So what is the policy here? Should has_line_minor() test if computing
 inverses is possible before handing the problem down?

 ----
 New commits:
 
||[http://git.sagemath.org/sage.git/commit/?id=dd3ad7d2943d03db3ad8e1001362704ece1ed77b
 dd3ad7d]||{{{Added an exception clause to catch a NonImplementedError
 below LinearMatroid.has_minor()}}}||

--
Ticket URL: <http://trac.sagemath.org/ticket/15295#comment:4>
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 http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to