Opened 11 years ago

Closed 8 years ago

#8918 closed defect (duplicate)

Strange behavior for Permutation()

Reported by: lvendramin Owned by: sage-combinat
Priority: minor Milestone: sage-duplicate/invalid/wontfix
Component: combinatorics Keywords:
Cc: jbandlow Merged in:
Authors: Reviewers: Luis Felipe Tabera Alonso
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

See these examples: 

sage: Permutation([1,2,3])
[1, 2, 3]
sage: Permutation([1,2,3,1])
[1, 2, 3, 1]
sage: [1,2,3] in Permutations()
True
sage: [1,2,3,1] in Permutations()
False
sage: Permutation([1,2,3,1]) in Permutations()
True

Change History (6)

comment:1 follow-up: Changed 11 years ago by lftabera

Yes, this is an error

[1,2,3,1] dos not define a permutation, so the method should raise an error. See also #9831

comment:2 in reply to: ↑ 1 Changed 8 years ago by knsam

I think, this in fact can be closed as a duplicate of the ticket you mention (#9831).

Replying to lftabera:

Yes, this is an error

[1,2,3,1] dos not define a permutation, so the method should raise an error. See also #9831

comment:3 Changed 8 years ago by knsam

  • Milestone changed from sage-5.7 to sage-duplicate/invalid/wontfix

comment:4 Changed 8 years ago by tscrim

  • Status changed from new to needs_review

One of a few tickets like this. Bonus points if you can find them all.

Also, don't forget to set this to needs_review. Thanks.

comment:5 Changed 8 years ago by tscrim

  • Status changed from needs_review to positive_review

comment:6 Changed 8 years ago by jdemeyer

  • Authors lvendramin deleted
  • Resolution set to duplicate
  • Reviewers set to Luis Felipe Tabera Alonso
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.