#13862: Split graph_generators into several files
---------------------------------+------------------------------------------
Reporter: ncohen | Owner: jason, ncohen, rlm
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-5.6
Component: graph theory | Resolution:
Keywords: | Work issues:
Report Upstream: N/A | Reviewers:
Authors: Nathann Cohen | Merged in:
Dependencies: | Stopgaps:
---------------------------------+------------------------------------------
Changes (by ncohen):
* status: new => needs_review
Old description:
> This tickets addresses the fact that Jeroen is not happy with the length
> of `graph_generators.py` `:-P`
>
> It splits the file into new ones, added in the graph/generators/ folder,
> according to the classifiation of the methods that appears in the
> module's documentation.
>
> APPLY:
> * [attachment:trac_13862.patch]
> * [attachment:trac_13862-cleaning_and_moving.patch]
> * [attachment:trac_13862-doc.patch]
> * [attachment:trac_13862-code_reformatting.patch]
New description:
This tickets addresses the fact that Jeroen is not happy with the length
of `graph_generators.py` `:-P`
It splits the file into new ones, added in the graph/generators/ folder,
according to the classifiation of the methods that appears in the module's
documentation.
APPLY:
* [attachment:trac_13862.patch]
* [attachment:trac_13862-cleaning_and_moving.patch]
* [attachment:trac_13862-doc.patch]
* [attachment:trac_13862-code_reformatting.patch]
* [attachment:trac_13862-moves_3_constructors.patch]
--
Comment:
What this patch does ([attachment:trac_13862-moves_3_constructors.patch]):
* Moves `CompleteGraph`, `CompleteBipartiteGraph` and
`CompleteMultipartiteGraph` from `families.py` to `basic.py`
* Adds `CompleteMultipartiteGraph` to the lists, as it did not appears
before `O_o`
And nooooow, this ticket is ready to be reviewed !!! `:-P`
Nathann
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13862#comment:8>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
