Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fusion Ring - Rmatrix #30257

Closed
dwbump mannequin opened this issue Jul 30, 2020 · 19 comments
Closed

Fusion Ring - Rmatrix #30257

dwbump mannequin opened this issue Jul 30, 2020 · 19 comments

Comments

@dwbump
Copy link
Mannequin

dwbump mannequin commented Jul 30, 2020

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.)

CC: @tscrim @sagetrac-sage-combinat @dwbump @willieab @physicstravels

Component: combinatorics

Keywords: FusionRing, R-matrix

Author: Daniel Bump, Guillermo Aboumrad

Branch/Commit: db68965

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/30257

@dwbump dwbump mannequin added this to the sage-9.2 milestone Jul 30, 2020
@dwbump dwbump mannequin added c: combinatorics labels Jul 30, 2020
@dwbump
Copy link
Mannequin Author

dwbump mannequin commented Jul 30, 2020

Branch: public/fusion-rmatrix-30257

@dwbump
Copy link
Mannequin Author

dwbump mannequin commented Jul 30, 2020

Commit: 637b3fd

@dwbump
Copy link
Mannequin Author

dwbump mannequin commented Jul 30, 2020

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

@dwbump dwbump mannequin added the t: enhancement label Jul 30, 2020
@dwbump
Copy link
Mannequin Author

dwbump mannequin commented Jul 31, 2020

comment:3

Rowell, Stong and Wang was corrected in arXiv version 4 and Example 5.4.5 agrees with our code in the corrected version.

@dwbump

This comment has been minimized.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 5, 2020

Changed commit from 637b3fd to e42ca5d

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 5, 2020

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

f22cac0documentation work
e42ca5ddocumentation work

@dwbump
Copy link
Mannequin Author

dwbump mannequin commented Aug 5, 2020

comment:6

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.

@dwbump

This comment has been minimized.

@dwbump dwbump mannequin added the s: needs review label Aug 5, 2020
@dwbump

This comment has been minimized.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 5, 2020

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

4165a9cpyflakes and pycodestyle requested revisions

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 5, 2020

Changed commit from e42ca5d to 4165a9c

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 5, 2020

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

ad0a2f3more pyflakes revisions

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 5, 2020

Changed commit from 4165a9c to ad0a2f3

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 8, 2020

Changed commit from ad0a2f3 to db68965

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 8, 2020

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.

@tscrim
Copy link
Collaborator

tscrim commented Aug 8, 2020

comment:12

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.

@tscrim
Copy link
Collaborator

tscrim commented Aug 8, 2020

Reviewer: Travis Scrimshaw

@dwbump dwbump mannequin added s: positive review and removed s: needs review labels Aug 9, 2020
@vbraun
Copy link
Member

vbraun commented Aug 12, 2020

Changed branch from public/fusion-rmatrix-30257 to db68965

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants