#14967: Bug with DiGraph
--------------------------------+--------------------------
       Reporter:  mercatp       |         Owner:
           Type:  defect        |        Status:  new
       Priority:  major         |     Milestone:  sage-5.11
      Component:  graph theory  |    Resolution:
       Keywords:  graph         |     Merged in:
        Authors:  Paul Mercat   |     Reviewers:
Report Upstream:  N/A           |   Work issues:
         Branch:                |  Dependencies:
       Stopgaps:                |
--------------------------------+--------------------------

Comment (by mercatp):

 Ok, I understand.
 So the test {{{if u in IntegerRing()}}} is not a good idea, it would be
 better to impose u to be of type int for example :
 {{{#!python
 if type(u) == int:
    u_int = u
 else:
    return -1
 }}}
 This should be faster.
 And it's better to impose a strict type for the number, because an 1 in a
 given number field is not the same that an 1 in an other number field for
 example (we can imagine a graph with differents vertex 1, although it's
 weird).

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