#15603: "immutable=True" for Graph/Digraph __init__ and copy()
-------------------------+-------------------------------------------------
Reporter: | Owner:
ncohen | Status: needs_review
Type: | Milestone: sage-6.1
enhancement | Resolution:
Priority: major | Merged in:
Component: graph | Reviewers:
theory | Work issues:
Keywords: | Commit:
Authors: | 7860f3997ae9ecd954f0696025fbb63e0219e78b
Nathann Cohen | Stopgaps:
Report Upstream: N/A |
Branch: |
u/ncohen/15603 |
Dependencies: |
#15278 |
-------------------------+-------------------------------------------------
Comment (by SimonKing):
Things that need to be done (in a review commit): In some place you
explain the `immutable=...` optional argument, but in the example you use
`data_structure=...` only. In some other place, you do this change:
{{{
#!diff
If the ``data_structure`` is equal to ``"static_sparse"``, then an
immutable graph results. Note that this does not use the NetworkX
data
structure::
- sage: G_imm = Graph(g, data_structure="static_sparse")
- sage: H_imm = Graph(g, data_structure="static_sparse")
+ sage: G_imm = Graph(g, immutable=True)
+ sage: H_imm = Graph(g, immutable=True)
sage: G_imm == H_imm == G == H
True
}}}
Hence, the text is about data_structure, but you remove it from the test.
--
Ticket URL: <http://trac.sagemath.org/ticket/15603#comment:10>
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.
