#20499: Graphs: docstring of _add_ conflicts with function
--------------------------------+--------------------------
       Reporter:  jmantysalo    |        Owner:
           Type:  defect        |       Status:  needs_info
       Priority:  major         |    Milestone:  sage-7.2
      Component:  graph theory  |   Resolution:
       Keywords:                |    Merged in:
        Authors:                |    Reviewers:
Report Upstream:  N/A           |  Work issues:
         Branch:                |       Commit:
   Dependencies:                |     Stopgaps:
--------------------------------+--------------------------

Comment (by jmantysalo):

 Replying to [comment:7 mcognetta]:

 > In the end, I think that relabeling to integers is best. If people know
 they need to preserve vertex names, they can take precautions beforehand.

 OK for me. Then it would be like `.disjoint_union(other,
 labels=’integers’)`. Current behaviour would be maintained, so there is no
 need for a deprecation. Should `disjoint_union` have an option to get
 really direct union (and raise an exception it graphs have common
 vertices)? If not, I will add an example like we now have for disjoint
 union of posets.

 And then there is the naming thing.

 But first I will wait for three days to see if there will be opinions
 against this.

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