Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#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:


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 

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)

trac_7149-delete-graphtheory.patch (9.4 KB) - added by jhpalmieri 11 years ago.

Download all attachments as: .zip

Change History (5)

Changed 11 years ago by jhpalmieri

comment:1 Changed 11 years ago by was

  • 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

thanks for noticing this!

merged in 4.2.1.rc1

comment:2 Changed 11 years ago by jason

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 jhpalmieri

We can reopen #6774 or open a new ticket -- I'm not sure which is better. In any case, we should make sure that the most recent version of the graph theory tour is included (combining at least #6774 and #6952 -- are there any other relevant tickets?).

comment:4 Changed 11 years ago by mhansen

  • 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.