Ticket #3626 (closed defect: fixed)

Opened 3 months ago

Last modified 3 months ago

[with patch, positive review] Graph.set_boundary only takes lists

Reported by: boothby Assigned to: boothby
Priority: minor Milestone: sage-3.0.6
Component: graph theory Keywords:
Cc: rlm

Description 

sage: G = Graph("George")
sage: G.set_boundary(set([1,2,3]))
sage: G.get_boundary()
[]

... which makes sense, given the code... 

    def set_boundary(self, boundary):
        ...
        if isinstance(boundary,list):
            self._boundary = boundary

    def set_embedding(self, embedding):
        ...

Attachments

3626_graphboundary.patch (0.9 kB) - added by boothby on 07/09/2008 11:53:37 AM.

Change History

07/09/2008 11:53:37 AM changed by boothby

  • attachment 3626_graphboundary.patch added.

07/09/2008 01:56:37 PM changed by mhansen

  • summary changed from Graph.set_boundary only takes lists to [with patch, needs review] Graph.set_boundary only takes lists.

07/11/2008 03:33:18 PM changed by boothby

  • cc set to rlm.

07/12/2008 09:51:26 AM changed by rlm

  • summary changed from [with patch, needs review] Graph.set_boundary only takes lists to [with patch, positive review] Graph.set_boundary only takes lists.

+1

07/15/2008 05:42:12 PM changed by mabshoff

  • status changed from new to closed.
  • resolution set to fixed.

Merged in Sage 3.0.6.alpha0