#15494: Deprecate the _boundary parameter
----------------------------+---------------------------------
   Reporter:  ncohen        |            Owner:
       Type:  defect        |           Status:  new
   Priority:  major         |        Milestone:  sage-5.13
  Component:  graph theory  |         Keywords:
  Merged in:                |          Authors:  Nathann Cohen
  Reviewers:                |  Report Upstream:  N/A
Work issues:                |           Branch:
     Commit:                |     Dependencies:
   Stopgaps:                |
----------------------------+---------------------------------
 From the documentation (`Graph.get_boundary?` or `Graph.set_boundary?`)
 there is just no way to know what it does. So because of that probably
 nobody uses it. Or they guessed the purpose in some way I can't guess
 myself.

 Then there are things like linear-time operations in functions like
 `.delete_vertex` (noticed by #15278).

 {{{
 self._boundary = [v for v in self._boundary if v != vertex]
 }}}

 Sooooooooo unless somebody complains and is willing to write some
 documentation for this parameter, I think it'd be nice to deprecate then
 remove it.

 Nathan

--
Ticket URL: <http://trac.sagemath.org/ticket/15494>
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