Opened 9 years ago

Closed 9 years ago

#8640 closed defect (fixed)

Add BipartiteGraph to the documentation

Reported by: ncohen Owned by: mvngu
Priority: major Milestone: sage-4.4.2
Component: documentation Keywords:
Cc: rhinton Merged in: sage-4.4.2.rc0
Authors: Minh Van Nguyen Reviewers: David Joyner
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by mvngu)

For the moment, Sage's documentation does not include the BipartiteGraph? class.

For the moment, patches #8329 #8421 and #8425 are still to be merged, and waiting for them to be is a lazy way to avoid conflicts :-)

Prerequisites: #8819, #8831

Apply patches in this order:

  1. #8819
  2. #8831
  3. trac_8640-bipartite.patch

Attachments (1)

trac_8640-bipartite.patch (39.7 KB) - added by mvngu 9 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 9 years ago by mvngu

Adding the bipartite graph class to the reference manual results in the following warnings:

/dev/shm/mvngu/sandbox/sage-4.4.1.alpha2-8640-bipartite/local/lib/python2.6/site-packages/sage/graphs/bipartite_graph.py:docstring of sage.graphs.bipartite_graph:3: (ERROR/3) Unexpected indentation.
/dev/shm/mvngu/sandbox/sage-4.4.1.alpha2-8640-bipartite/local/lib/python2.6/site-packages/sage/graphs/bipartite_graph.py:docstring of sage.graphs.bipartite_graph.BipartiteGraph:25: (WARNING/2) Literal block expected; none found.
/dev/shm/mvngu/sandbox/sage-4.4.1.alpha2-8640-bipartite/local/lib/python2.6/site-packages/sage/graphs/bipartite_graph.py:docstring of sage.graphs.bipartite_graph.BipartiteGraph.delete_vertex:7: (WARNING/2) Inline interpreted text or phrase reference start-string without end-string.
/dev/shm/mvngu/sandbox/sage-4.4.1.alpha2-8640-bipartite/local/lib/python2.6/site-packages/sage/graphs/bipartite_graph.py:docstring of sage.graphs.bipartite_graph:1: (ERROR/3) Unexpected indentation.
/dev/shm/mvngu/sandbox/sage-4.4.1.alpha2-8640-bipartite/local/lib/python2.6/site-packages/sage/graphs/bipartite_graph.py:docstring of sage.graphs.bipartite_graph.BipartiteGraph.load_afile:5: (WARNING/2) Block quote ends without a blank line; unexpected unindent.
/dev/shm/mvngu/sandbox/sage-4.4.1.alpha2-8640-bipartite/local/lib/python2.6/site-packages/sage/graphs/bipartite_graph.py:docstring of sage.graphs.bipartite_graph:8: (ERROR/3) Unexpected indentation.
/dev/shm/mvngu/sandbox/sage-4.4.1.alpha2-8640-bipartite/local/lib/python2.6/site-packages/sage/graphs/bipartite_graph.py:docstring of sage.graphs.bipartite_graph.BipartiteGraph.load_afile:14: (WARNING/2) Block quote ends without a blank line; unexpected unindent.
/dev/shm/mvngu/sandbox/sage-4.4.1.alpha2-8640-bipartite/local/lib/python2.6/site-packages/sage/graphs/bipartite_graph.py:docstring of sage.graphs.bipartite_graph:17: (ERROR/3) Unexpected indentation.
/dev/shm/mvngu/sandbox/sage-4.4.1.alpha2-8640-bipartite/local/lib/python2.6/site-packages/sage/graphs/bipartite_graph.py:docstring of sage.graphs.bipartite_graph.BipartiteGraph.reduced_adjacency_matrix:16: (WARNING/2) Definition list ends without a blank line; unexpected unindent.
/dev/shm/mvngu/sandbox/sage-4.4.1.alpha2-8640-bipartite/local/lib/python2.6/site-packages/sage/graphs/bipartite_graph.py:docstring of sage.graphs.bipartite_graph.BipartiteGraph:9: (ERROR/3) Unexpected indentation.
/dev/shm/mvngu/sandbox/sage-4.4.1.alpha2-8640-bipartite/local/lib/python2.6/site-packages/sage/graphs/bipartite_graph.py:docstring of sage.graphs.bipartite_graph.BipartiteGraph.save_afile:5: (WARNING/2) Definition list ends without a blank line; unexpected unindent.
/dev/shm/mvngu/sandbox/sage-4.4.1.alpha2-8640-bipartite/local/lib/python2.6/site-packages/sage/graphs/bipartite_graph.py:docstring of sage.graphs.bipartite_graph.BipartiteGraph.save_afile:10: (WARNING/2) Definition list ends without a blank line; unexpected unindent.

These warnings must be resolved before we could add the bipartite graph class to the reference manual.

Changed 9 years ago by mvngu

comment:2 Changed 9 years ago by mvngu

  • Authors set to Minh Van Nguyen
  • Description modified (diff)
  • Status changed from new to needs_review

comment:3 Changed 9 years ago by mvngu

Changes in the patch include:

  • Resolve all the warnings.
  • Clean-ups in accordance with PEP 008

comment:4 Changed 9 years ago by wdj

  • Status changed from needs_review to positive_review

Applies to 4.4.2.a0 and build fine. Looks good to me.

comment:5 Changed 9 years ago by mvngu

  • Merged in set to sage-4.4.2.rc0
  • Resolution set to fixed
  • Reviewers set to David Joyner
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.