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

WeylGroup.subgroup doesn't create a subset #24393

Closed
vit-tucek mannequin opened this issue Dec 16, 2017 · 6 comments
Closed

WeylGroup.subgroup doesn't create a subset #24393

vit-tucek mannequin opened this issue Dec 16, 2017 · 6 comments

Comments

@vit-tucek
Copy link
Mannequin

vit-tucek mannequin commented Dec 16, 2017

The following code

W = WeylGroup("A2", prefix="s")
s = W.simple_reflections()
H = W.subgroup([s[2]])
s[2] in list(H)

returns False. One workaround is s[2] in list(W(x) for x in H).

CC: @sagetrac-sage-combinat

Component: combinatorics

Reviewer: Travis Scrimshaw, Simon Brandhorst

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

@vit-tucek vit-tucek mannequin added this to the sage-8.2 milestone Dec 16, 2017
@vit-tucek vit-tucek mannequin changed the title WeylGroup.subgroup is not a subset WeylGroup.subgroup doesn't create a subset Dec 16, 2017
@vit-tucek
Copy link
Mannequin Author

vit-tucek mannequin commented Dec 16, 2017

comment:2

Seems like a duplicate of #19010

@tscrim
Copy link
Collaborator

tscrim commented Dec 18, 2017

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Dec 18, 2017

comment:3

I agree that this is essentially a duplicate of #19010 as the root cause is the same:

sage: W.has_coerce_map_from(H)
False

Every subgroup should have a coercion into its ambient group.

@tscrim tscrim removed this from the sage-8.2 milestone Dec 18, 2017
@simonbrandhorst
Copy link

comment:4

indeed.

@simonbrandhorst
Copy link

Changed reviewer from Travis Scrimshaw to Travis Scrimshaw, Simon Brandhorst

@embray
Copy link
Contributor

embray commented Feb 26, 2019

comment:6

Presuming these are all correctly reviewed as either duplicate, invalid, or wontfix.

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