Opened 8 years ago

Closed 7 years ago

#14863 closed defect (fixed)

permutation from major code returns the wrong permutation for the empty permutation

Reported by: stumpc5 Owned by: sage-combinat
Priority: major Milestone: sage-5.12
Component: combinatorics Keywords: permutation, major index, FindStat
Cc: Merged in: sage-5.12.beta0
Authors: Christian Stump Reviewers: Nathann Cohen
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description 

sage: from sage.combinat.permutation import from_major_code
sage: from_major_code([])                                  
[0]

Attachments (1)

trac_14863-perm_from_maj_code-cs.patch​ (1014 bytes) - added by stumpc5 8 years ago.

Download all attachments as: .zip

Change History (4)

comment:1 Changed 8 years ago by stumpc5

  • Status changed from new to needs_review

Changed 8 years ago by stumpc5

comment:2 Changed 8 years ago by ncohen

  • Reviewers set to Nathann Cohen
  • Status changed from needs_review to positive_review

Helloooooo !!

I don't get why you don't just return Permutation([]) in that case, but well.. That's more or less the same :-P

Nathann

comment:3 Changed 7 years ago by jdemeyer

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