Opened 14 months ago
Closed 13 months ago
#32283 closed defect (fixed)
Fix cellular_involution for tensor product of cellular algebras
Reported by:  Travis Scrimshaw  Owned by:  

Priority:  major  Milestone:  sage9.5 
Component:  algebra  Keywords:  cellular 
Cc:  Frédéric Chapoton  Merged in:  
Authors:  Travis Scrimshaw  Reviewers:  Frédéric Chapoton 
Report Upstream:  N/A  Work issues:  
Branch:  2c6225f (Commits, GitHub, GitLab)  Commit:  2c6225fd071ec6d983b1b2687f4ad22206d21e46 
Dependencies:  Stopgaps: 
Description
There is a serious bug in the implementation of cellular_involution
for a tensor product of cellular algebras that is also expressed as the cellular basis.
Change History (8)
comment:1 Changed 14 months ago by
Branch:  → public/categories/fix_cellular_involution32283 

Commit:  → ebad0b001771d87d399464e6fb0e962e099c9578 
Status:  new → needs_review 
comment:3 Changed 14 months ago by
Ah, crap. I forgot to change that back to x
. I will correct this tomorrow when I get to my office.
comment:4 Changed 14 months ago by
Commit:  ebad0b001771d87d399464e6fb0e962e099c9578 → 2c6225fd071ec6d983b1b2687f4ad22206d21e46 

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
2c6225f  Fixing cellular_involution for tensor of finitedimensional cellular algebras.

comment:6 Changed 14 months ago by
Reviewers:  → Frédéric Chapoton 

Status:  needs_review → positive_review 
ok
comment:7 Changed 14 months ago by
Milestone:  sage9.4 → sage9.5 

comment:8 Changed 13 months ago by
Branch:  public/categories/fix_cellular_involution32283 → 2c6225fd071ec6d983b1b2687f4ad22206d21e46 

Resolution:  → fixed 
Status:  positive_review → closed 
Note: See
TracTickets for help on using
tickets.
There is currently no implementation that this case would cover, but it is conceivable that it might occur. However, because of this, it is not something I can easily add a doctest for.
New commits:
Fixing cellular_involution for tensor of finitedimensional cellular algebras.