Ticket #11414 (closed defect: duplicate)

Opened 2 years ago

Last modified 18 months ago

CartesianProduct should return a list of tuples

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

Description (last modified by pdehaye) (diff) 

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

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

Change History

comment:1 Changed 2 years ago by pdehaye

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

comment:2 Changed 2 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)

Changed 2 years ago by pdehaye

comment:3 Changed 2 years ago by pdehaye

  • Status changed from needs_review to needs_work

comment:4 Changed 2 years ago by pdehaye

  • Owner changed from sage-combinat to pdehaye

comment:5 Changed 2 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 18 months ago by roed

  • Status changed from needs_work to positive_review

comment:7 Changed 18 months ago by jdemeyer

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