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.

Sign up for GitHub

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

Jump to bottom

Docstring edits to #35501 #35569

Closed
wants to merge 28 commits into from
Closed

Docstring edits to #35501 #35569

wants to merge 28 commits into from

Conversation

dwbump
Copy link
Contributor

@dwbump dwbump commented Apr 25, 2023

📚 Description

📝 Checklist

  • The title is concise, informative, and self-explanatory.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • [x ] I have created tests covering the changes.
  • [ x] I have updated the documentation accordingly.

⌛ Dependencies

#35501

dwbmscz and others added 28 commits March 4, 2023 10:57
@dwbmscz
fusion_double
58aef8d
@dwbmscz
char_Nk_ij copied from enqiLi with minor changes
f4bc7bd
@dwbmscz
New method char_N_ij adapting Wenqi Li's code. Docstring revisions
264df99
@dwbump
import of older FMatrix code from sage trac commit 1b99dcc temporaril…
20e2b46 
…y into fusion_double.py
@dwbmscz
work on provisional F-matrix code
7cb7da0
@dwbmscz
improvements to fusion_double
bfac555
@dwbmscz
working F-matrix code
e8685d5
@dwbmscz
Doc revision: fusion_double
bfb3e18
@dwbmscz
Correct name of Shawn Cui in existing reference [CHW2015]
729d4af
@dwbmscz
docstring revisions
549f4b3
@dwbump
Merge branch 'develop' into fusion-double
8054eec
@dwbmscz
pycodestyle asked changes
5f0872e
@dwbmscz
changes asked by relint
f89fc1d
@dwbmscz
a few more minor changes
4944d80
@dwbmscz
a few more doctests
149581a
@dwbmscz
doctests for most methods in fusion_double
2e016eb
@dwbump
remove trailing whitespace in fusion_double.py
637d883
@dwbump
Merge branch 'develop' into fusion-double
d36a9de
@dwbmscz
100 percent coverage in fusion_double.py
2d7c8b8
@dwbmscz
Merge branch 'fusion-double' of github.com:dwbump/sage into fusion-do…
d758b43 
…uble
@dwbmscz
remove spurious import of FMatrix1
abbd5f7
@dwbmscz
minor fixes in fusion_double.py
75131ce
@dwbmscz
missing EXAMPLES::
62a12e1
@dwbmscz
bugfix in s_matrix
05fbc82
@willieab
first stab at re-factoring fusion ring code. doc tests pass except fo…
c9c7761 
…r a couple issues with get_order and FusionRingFromWCR.__classcall__. Need some documentation/explanation in FusionRing constructor
@willieab
updating concrete class names and FusionRing usage to be consistent w…
a0ed4c3 
…ith original FusionRing. 10 get_order /__classcall__ doctests still failing...
@dwbmscz
edit docstring to FusionRing
a0b7d6a
@dwbmscz
port some of the revisions for sagemath#35389 to sagemath#35501
4c18d8a
@dwbump dwbump mentioned this pull request Apr 25, 2023
Draft
4 tasks
@dwbump dwbump closed this Apr 27, 2023
@dwbump
Copy link
Contributor Author

dwbump commented Apr 27, 2023

According to comments on #35501 this should have been a PR against Willie's branch, not sagemath, so I am closing this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dwbump @dwbmscz @willieab