Opened 10 years ago

Closed 9 years ago

#11414 closed defect (duplicate)

CartesianProduct should return a list of tuples

Reported by: pdehaye Owned by: pdehaye
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: combinatorics Keywords:
Cc: Merged in:
Authors: Reviewers: David Roe
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by pdehaye)

sage: list(CartesianProduct([1,2],[3,4]))
[[1, 3], [1, 4], [2, 3], [2, 4]]

Sadly, this is not the case.

correction: seems to have been fixed on its own...

Attachments (1)

trac_11414-CartesianProduct_yields_tuples-pod.patch (9.8 KB) - added by pdehaye 10 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 10 years ago by pdehaye

  • Description modified (diff)
  • Priority changed from minor to major
  • Summary changed from PartitionTuples should return a tuple to CartesianProduct should return a list of tuples

comment:2 Changed 10 years ago by pdehaye

  • Cc hivert mhansen added
  • Status changed from new to needs_review

i realized this problem while fixing #11412, so this needs to be applied on top of the patch over there. (i m not sure i proceeded as was best)

comment:3 Changed 10 years ago by pdehaye

  • Status changed from needs_review to needs_work

comment:4 Changed 10 years ago by pdehaye

  • Owner changed from sage-combinat to pdehaye

comment:5 Changed 10 years ago by pdehaye

  • Cc hivert mhansen removed
  • Description modified (diff)
  • Milestone changed from sage-4.7.1 to sage-duplicate/invalid/wontfix

comment:6 Changed 9 years ago by roed

  • Status changed from needs_work to positive_review

comment:7 Changed 9 years ago by jdemeyer

  • Resolution set to duplicate
  • Reviewers set to David Roe
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.