#7149 closed defect (fixed)
[with patch, positive review] tutorial: delete the graph theory section
Reported by: | jhpalmieri | Owned by: | jhpalmieri |
---|---|---|---|
Priority: | blocker | Milestone: | sage-4.1.2 |
Component: | documentation | Keywords: | |
Cc: | Merged in: | sage-4.1.2.rc1 | |
Authors: | John Palmieri | Reviewers: | Jason Grout |
Report Upstream: | Work issues: | ||
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
From sage-devel:
As of version 4.1.2.alpha2, the Sage tutorial has a section containing a guided tour of graph theory in Sage. In principle, this is a good idea, but the execution is severely flawed: about 2/3 of the tour focuses on the methods "g.max_matching()", "g.edge_coloring()", and "g.vertex_coloring()". There are no such methods in Sage. (The relevant doctests were never executed because of the issue discussed at trac ticket #6572.) Since the tutorial ought to be one of the first pieces of documentation people use, this situation is disastrous. I suggest that before we release 4.1.2, we delete this part of the tutorial until the file is fixed. (Alternatively, we could delete 2/3 of the file, but that might make it a bit short on substance.)
See #6572 for some related issues.
Attachments (1)
Change History (5)
Changed 11 years ago by
comment:1 Changed 11 years ago by
- Resolution set to fixed
- Status changed from new to closed
- Summary changed from [with patch, needs review] tutorial: delete the graph theory section to [with patch, positive review] tutorial: delete the graph theory section
comment:2 Changed 11 years ago by
This was from #6774. I gave it a positive review, pending the merging of the tickets with the necessary functionality, since I did not have those tickets merged in my tree yet. Minh merged it after the doctests passed (I guess assuming that the functions had been merged then), so it does look like it was a problem of the doctests not actually being run (manually or otherwise!).
Thanks for catching this; sorry for the bother!
We should probably reopen #6774 now, for when the functionality is merged into Sage.
comment:3 Changed 11 years ago by
comment:4 Changed 11 years ago by
- Merged in set to sage-4.1.2.rc1
- Milestone changed from sage-4.2 to sage-4.1.2
- Reviewers set to Jason Grout
Note: See
TracTickets for help on using
tickets.
thanks for noticing this!
merged in 4.2.1.rc1