Opened 10 years ago

Closed 9 years ago

#9230 closed defect (fixed)

Broken docstrings in Travelling Salesman Problem

Reported by: ncohen Owned by: jason, mvngu, ncohen, rlm
Priority: major Milestone: sage-4.5
Component: graph theory Keywords:
Cc: mvngu Merged in: sage-4.5.alpha1
Authors: Nathann Cohen Reviewers: Minh Van Nguyen
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by mvngu)

Still those annoying default {} instead of None. This patch was not merged when the fix for {} was written ;

Prerequisite: #8781

Attachments (1)

trac_9230.patch (1.6 KB) - added by ncohen 10 years ago.

Download all attachments as: .zip

Change History (4)

Changed 10 years ago by ncohen

comment:1 Changed 10 years ago by ncohen

  • Status changed from new to needs_review

Here it is !

Nathann

comment:2 Changed 10 years ago by mvngu

  • Authors set to Nathann Cohen
  • Description modified (diff)
  • Reviewers set to Minh Van Nguyen
  • Status changed from needs_review to positive_review

Looks good. The relevant failure was discovered when testing #8781, so I'm making #8781 a prerequisite of the current ticket. That way, if the current ticket is merged, then #8781 must first be merged.

comment:3 Changed 9 years ago by rlm

  • Merged in set to sage-4.5.alpha1
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.