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

Tensor product of crystals does not handle -inf for epsilon/phi correctly #18469

Closed
tscrim opened this issue May 22, 2015 · 8 comments
Closed

Comments

@tscrim
Copy link
Collaborator

tscrim commented May 22, 2015

This does not work:

sage: E1 = crystals.elementary.B(['A',2], 1)
sage: E2 = crystals.elementary.B(['A',2], 2)
sage: T = crystals.TensorProduct(E1, E2)
sage: x = T(E1.module_generators[0], E2.module_generators[0]); x
[0, 0]
sage: x.f(1)
[-1, 0]
sage: x.e(1)
[1, 0]

The reason is because we try and be clever with how we compute the _sig and do an epsilon_(k) - phi_i(k-1). If one of these (in particular, the phi) is -inf, then the computation breaks down. Instead, we have to bite the bullet and do the partial sums of the weight pairings to make sure we cover these cases.

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

Component: combinatorics

Keywords: crystals, tensor product

Author: Travis Scrimshaw

Branch/Commit: 9a24ea0

Reviewer: Ben Salisbury

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

@tscrim
Copy link
Collaborator Author

tscrim commented May 22, 2015

Commit: 211f611

@tscrim
Copy link
Collaborator Author

tscrim commented May 22, 2015

@tscrim
Copy link
Collaborator Author

tscrim commented May 22, 2015

New commits:

211f611Fixing _sig for epsilon/phi returning -inf.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 22, 2015

Changed commit from 211f611 to 9a24ea0

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 22, 2015

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

9a24ea0Fixing _sig for epsilon/phi returning -inf.

@bsalisbury1
Copy link

comment:3

Thanks, Travis.

@bsalisbury1
Copy link

Reviewer: Ben Salisbury

@vbraun
Copy link
Member

vbraun commented May 23, 2015

Changed branch from public/crystals/fix_tensor_product-18469 to 9a24ea0

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

3 participants