Opened 10 months ago

Closed 9 months ago

#30257 closed enhancement (fixed)

Fusion Ring - Rmatrix

Reported by: bump Owned by:
Priority: major Milestone: sage-9.2
Component: combinatorics Keywords: FusionRing, R-matrix
Cc: tscrim, sage-combinat, bump, gh-willieab, gh-physicstravels 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:

Status badges

Description (last modified by bump)

The FusionRing method captures much of the structure of a modular tensor category. One aspect that it does not capture is the R-matrix, 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 bump

  • Branch set to public/fusion-rmatrix-30257
  • Commit set to 637b3fda6b3d6929be60316232327a741452773c

New commits:

f498480new method rmatrix
6344585bugfix in q_dimension
ca2562cMerge branch 'cyclotomic' into rmatrix
f15e39ddoctest
39f894dbugfix: don't reduce twist
5fe4524use theta method in a few places
637b3fdwork on documentation

comment:2 Changed 10 months ago by bump

  • Type changed from PLEASE CHANGE to enhancement

comment:3 Changed 9 months ago by bump

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 bump

  • Description modified (diff)

comment:5 Changed 9 months ago by git

  • Commit changed from 637b3fda6b3d6929be60316232327a741452773c to e42ca5d9dc8e5cf62f01caf31d0a636da76f200c

Branch pushed to git repo; I updated commit sha1. New commits:

f22cac0documentation work
e42ca5ddocumentation work

comment:6 Changed 9 months ago by bump

This method has been tested a lot in the past couple of days in computing F-matrices (using the hexagon relations). I think it is reliable.

comment:7 Changed 9 months ago by bump

  • Description modified (diff)
  • Status changed from new to needs_review

comment:8 Changed 9 months ago by bump

  • Description modified (diff)

comment:9 Changed 9 months ago by git

  • Commit changed from e42ca5d9dc8e5cf62f01caf31d0a636da76f200c to 4165a9c2bb9a4b887c379a5d0255022d90212e21

Branch pushed to git repo; I updated commit sha1. New commits:

4165a9cpyflakes and pycodestyle requested revisions

comment:10 Changed 9 months ago by git

  • Commit changed from 4165a9c2bb9a4b887c379a5d0255022d90212e21 to ad0a2f38f53ca4cc22aedb82f3016b8cfd86736e

Branch pushed to git repo; I updated commit sha1. New commits:

ad0a2f3more pyflakes revisions

comment:11 Changed 9 months ago by git

  • Commit changed from ad0a2f38f53ca4cc22aedb82f3016b8cfd86736e to db689653dc4ba6ed4931cdbe652ac9ee3f5663d1

Branch pushed to git repo; I updated commit sha1. New commits:

eb257abMerge branch 'public/fusion-rmatrix-30257' of git://trac.sagemath.org/sage into public/fusion-rmatrix-30257
db68965Refactored rmatrix() -> r_matrix(). Some other reviewer changes.

comment:12 Changed 9 months ago by tscrim

  • 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 bump

  • Status changed from needs_review to positive_review

comment:14 Changed 9 months ago by vbraun

  • Branch changed from public/fusion-rmatrix-30257 to db689653dc4ba6ed4931cdbe652ac9ee3f5663d1
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.