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

Implement quantum group q-numbers #25067

Closed
tscrim opened this issue Mar 31, 2018 · 13 comments
Closed

Implement quantum group q-numbers #25067

tscrim opened this issue Mar 31, 2018 · 13 comments

Comments

@tscrim
Copy link
Collaborator

tscrim commented Mar 31, 2018

Split off of #15508 for better granularity and to use it to build other tickets. Implements [n]_q, [n]_q! and q-binomials that appear in quantum groups.

CC: @sagetrac-sage-combinat @darijgr @anneschilling @bsalisbury1 @AndrewAtLarge

Component: algebra

Keywords: quantum groups, q-analogs

Author: Travis Scrimshaw

Branch: e0893e0

Reviewer: Darij Grinberg

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

@tscrim tscrim added this to the sage-8.2 milestone Mar 31, 2018
@tscrim
Copy link
Collaborator Author

tscrim commented Mar 31, 2018

Commit: 67c6ba0

@tscrim
Copy link
Collaborator Author

tscrim commented Mar 31, 2018

@tscrim
Copy link
Collaborator Author

tscrim commented Mar 31, 2018

New commits:

67c6ba0Adding q-analogs used in quantum groups.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 31, 2018

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

e908d59Adding q-analogs used in quantum groups.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 31, 2018

Changed commit from 67c6ba0 to e908d59

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 31, 2018

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

e0893e0documentation improvements

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 31, 2018

Changed commit from e908d59 to e0893e0

@tscrim
Copy link
Collaborator Author

tscrim commented Mar 31, 2018

Reviewer: Darij Grinberg

@tscrim
Copy link
Collaborator Author

tscrim commented Mar 31, 2018

comment:4

Discussed with Darij and says good modulo his changes, which LGTM. So positive review. Thank you!

@AndrewMathas
Copy link
Member

comment:5

Sorry to be late to the party but are you aware of combinat/q_analogues.py? In particular, this module provides functions q_int, q_factorial and q_binomial (among others). I have not been through this ticket in detail but comparing the q_int function in this ticket appears to be less general than the existing function because, in q_analogues.py, q_int(n) allows n to be positive or negative whereas the q_int function in this ticket requires n to be non-negative. On the other hand, the quantum integer that the existing q_int returns is different to the one in this ticket.

I am not sure how much this ticket overlaps with the code that is already in sage but some one should check. At a minimum, I think that there should be some documentation links between this code and the pre-existing code and, arguably, the code should be merged so that we don't have code providing similar functionality in different places. Different function names, or a more general syntax that allows one to choose between the different quantum integers, should also be considered.

Annoyingly, the q_int defined in q_analogues.py is the natural quantum integer to use in Hecke algebras whereas the q_int defined in this ticket is what is required in the quantum group setting...so both are needed.

As I have not read through all of the code I won't set this back to needs work but my feeling is that it does need more work.

@AndrewMathas
Copy link
Member

comment:6

Oh, I see that there are already documentation links, so feel free to ignore the above. This said, adding a link from the documentation in combinat/q_analogues.py to the code in this ticket would also be useful.

@vbraun
Copy link
Member

vbraun commented May 12, 2018

Changed branch from public/quantum_groups/q_numbers-25067 to e0893e0

@fchapoton
Copy link
Contributor

Changed commit from e0893e0 to none

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

4 participants