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

molien_series() should not use GAP's VirtualCharacter #34854

Closed
dimpase opened this issue Dec 17, 2022 · 8 comments
Closed

molien_series() should not use GAP's VirtualCharacter #34854

dimpase opened this issue Dec 17, 2022 · 8 comments
Labels
c: group theory p: major / 3 t: bug
Milestone
sage-9.8

Comments

@dimpase
Copy link
Member

dimpase commented Dec 17, 2022

One can use GAP's PermutationCharacter() to avoid the troubles.

Currently we have a problem here:

sage: PG = PermutationGroup(["(1,2,3,4,5,6,7)","(5,6,7)"])
sage: PG.molien_series()
... weird error...

see the report in https://groups.google.com/d/msgid/sage-devel/42c6be87-614e-4a21-9273-91948f0c6216n%40googlegroups.com

CC: @tscrim

Component: group theory

Author: Dima Pasechnik

Branch/Commit: b7cf604

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/34854
@dimpase dimpase added this to the sage-9.8 milestone Dec 17, 2022
@dimpase
Copy link
Member Author

dimpase commented Dec 17, 2022

Author: Dima Pasechnik

@dimpase
Copy link
Member Author

dimpase commented Dec 17, 2022

Branch: trac/u/dimpase/groups/fix_molien

@dimpase
Copy link
Member Author

dimpase commented Dec 17, 2022

New commits:

b7cf604don't use virtual chars in Molien series computation

@dimpase
Copy link
Member Author

dimpase commented Dec 17, 2022

Changed branch from trac/u/dimpase/groups/fix_molien to u/dimpase/groups/fix_molien

@dimpase
Copy link
Member Author

dimpase commented Dec 17, 2022

Commit: b7cf604

@tscrim
Copy link
Collaborator

tscrim commented Dec 19, 2022

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Dec 19, 2022

comment:3

LGTM.

@vbraun
Copy link
Member

vbraun commented Jan 12, 2023

Changed branch from u/dimpase/groups/fix_molien to b7cf604

@vbraun vbraun closed this as completed in 6a8155e Jan 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: group theory p: major / 3 t: bug
Projects
None yet
Milestone
sage-9.8
Development

No branches or pull requests
3 participants
@vbraun @dimpase @tscrim