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

Inconsistent behavior with generator ordering in free Lie algebras #27069

Closed
ehaka mannequin opened this issue Jan 16, 2019 · 9 comments
Closed

Inconsistent behavior with generator ordering in free Lie algebras #27069

ehaka mannequin opened this issue Jan 16, 2019 · 9 comments

Comments

@ehaka
Copy link
Mannequin

ehaka mannequin commented Jan 16, 2019

This is an issue discovered due to #27018. Giving a free Lie algebra a set of generators that are not in lexicographic order causes graded_basis to output elements which are not recognized as basis elements of the Lie algebra due to different ordering behavior.

sage: Lxyz = LieAlgebra(QQ, ['x','y','z']).Lyndon()
sage: Lzxy = LieAlgebra(QQ, ['z','x','y']).Lyndon()
sage: Lxyz.graded_basis(2)
([x, y], [x, z], [y, z])
sage: Lzxy.graded_basis(2)
([z, x], [z, y], [x, y])
sage: x,y,z = Lxyz.gens()
sage: a,b,c = Lzxy.gens()
sage: a,b,c
(z, x, y)
sage: b.bracket(a)
[x, z]
sage: x.bracket(z)
[x, z]
sage: Lzxy._is_basis_element(b.leading_support(),a.leading_support())
True
sage: Lzxy._is_basis_element(a.leading_support(),b.leading_support())
False

Trying to compute brackets with these elements leads to unhelpful tracebacks, e.g.

sage: Lzxy[Lzxy.graded_basis(2)[0],Lzxy.graded_basis(1)[0]]
Traceback (most recent call last)
...
AttributeError: 'sage.algebras.lie_algebras.lie_algebra_element.Lie' object has no attribute '_right'

Component: algebra

Keywords: Lie algebras

Author: Eero Hakavuori, Travis Scrimshaw

Branch/Commit: a3f59d3

Reviewer: Travis Scrimshaw, Eero Hakavuori

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

@ehaka ehaka mannequin added this to the sage-8.6 milestone Jan 16, 2019
@ehaka ehaka mannequin added c: algebra labels Jan 16, 2019
@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 25, 2019

Changed commit from 4e27c06 to 45b4506

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 25, 2019

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

af8cac3Merge branch 'public/lie_elem_indexing-27018' of git://trac.sagemath.org/sage into public/lie_elem_indexing-27018
45b4506Using the word given by elements \NN to generate the free Lie algebra basis elements.

@tscrim
Copy link
Collaborator

tscrim commented Jan 25, 2019

Author: Eero Hakavuori, Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Jan 25, 2019

Reviewer: Travis Scrimshaw, Eero Hakavuori

@tscrim
Copy link
Collaborator

tscrim commented Jan 25, 2019

comment:2

Okay, this should fix it, as well as #27018.

@tscrim tscrim modified the milestones: sage-8.6, sage-8.7 Jan 25, 2019
@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 28, 2019

Changed commit from 45b4506 to a3f59d3

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 28, 2019

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

a3f59d3Small tweak to the documentation.

@ehaka
Copy link
Mannequin Author

ehaka mannequin commented Jan 28, 2019

comment:4

Everything seems to be working as it should now, thanks for the fix!

@ehaka ehaka mannequin added s: positive review and removed s: needs review labels Jan 28, 2019
@vbraun
Copy link
Member

vbraun commented Feb 4, 2019

Changed branch from public/lie_elem_indexing-27018 to a3f59d3

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