Opened 10 months ago
Closed 9 months ago
#30257 closed enhancement (fixed)
Fusion Ring  Rmatrix
Reported by:  bump  Owned by:  

Priority:  major  Milestone:  sage9.2 
Component:  combinatorics  Keywords:  FusionRing, Rmatrix 
Cc:  tscrim, sagecombinat, bump, ghwillieab, ghphysicstravels  Merged in:  
Authors:  Daniel Bump, Guillermo Aboumrad  Reviewers:  Travis Scrimshaw 
Report Upstream:  N/A  Work issues:  
Branch:  db68965 (Commits, GitHub, GitLab)  Commit:  db689653dc4ba6ed4931cdbe652ac9ee3f5663d1 
Dependencies:  Stopgaps: 
Description (last modified by )
The FusionRing
method captures much of the structure of a modular tensor category. One aspect that it does not capture is the Rmatrix, which is implemented in this ticket.
The results may be compared with calculations in Rowell, Stong and Wang (arXiv:0712.1377) and in Bonderson's thesis. Our results agree for several examples in Rowell, Stong and Wang. The reason for an apparent discrepancy in Example 5.4.5 was investigated. (See comment:3.)
Change History (14)
comment:1 Changed 10 months ago by
 Branch set to public/fusionrmatrix30257
 Commit set to 637b3fda6b3d6929be60316232327a741452773c
comment:2 Changed 10 months ago by
 Type changed from PLEASE CHANGE to enhancement
comment:3 Changed 9 months ago by
Rowell, Stong and Wang was corrected in arXiv version 4 and Example 5.4.5 agrees with our code in the corrected version.
comment:4 Changed 9 months ago by
 Description modified (diff)
comment:5 Changed 9 months ago by
 Commit changed from 637b3fda6b3d6929be60316232327a741452773c to e42ca5d9dc8e5cf62f01caf31d0a636da76f200c
comment:6 Changed 9 months ago by
This method has been tested a lot in the past couple of days in computing Fmatrices (using the hexagon relations). I think it is reliable.
comment:7 Changed 9 months ago by
 Description modified (diff)
 Status changed from new to needs_review
comment:8 Changed 9 months ago by
 Description modified (diff)
comment:9 Changed 9 months ago by
 Commit changed from e42ca5d9dc8e5cf62f01caf31d0a636da76f200c to 4165a9c2bb9a4b887c379a5d0255022d90212e21
Branch pushed to git repo; I updated commit sha1. New commits:
4165a9c  pyflakes and pycodestyle requested revisions

comment:10 Changed 9 months ago by
 Commit changed from 4165a9c2bb9a4b887c379a5d0255022d90212e21 to ad0a2f38f53ca4cc22aedb82f3016b8cfd86736e
Branch pushed to git repo; I updated commit sha1. New commits:
ad0a2f3  more pyflakes revisions

comment:11 Changed 9 months ago by
 Commit changed from ad0a2f38f53ca4cc22aedb82f3016b8cfd86736e to db689653dc4ba6ed4931cdbe652ac9ee3f5663d1
comment:12 Changed 9 months ago by
 Reviewers set to Travis Scrimshaw
I renamed rmatrix()
to r_matrix()
to be consistent with s_matrix()
and I think this is easier to read.
I renamed twist
parameter reduce
to reduced
to avoid a conflict with the reduce
builtin Python method.
I also made some minor changes to the docstrings.
If my changes are good, then positive review.
comment:13 Changed 9 months ago by
 Status changed from needs_review to positive_review
comment:14 Changed 9 months ago by
 Branch changed from public/fusionrmatrix30257 to db689653dc4ba6ed4931cdbe652ac9ee3f5663d1
 Resolution set to fixed
 Status changed from positive_review to closed
New commits:
new method rmatrix
bugfix in q_dimension
Merge branch 'cyclotomic' into rmatrix
doctest
bugfix: don't reduce twist
use theta method in a few places
work on documentation