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

SymmetricGroup(...).algebra(..., category=...) for consistency #18273

Closed
nthiery opened this issue Apr 22, 2015 · 7 comments
Closed

SymmetricGroup(...).algebra(..., category=...) for consistency #18273

nthiery opened this issue Apr 22, 2015 · 7 comments
Labels
c: algebra p: major / 3 t: bug
Milestone
sage-6.7

Comments

@nthiery
Copy link
Contributor

nthiery commented Apr 22, 2015

The generic algebra method takes a category option. This option is missing for SymmetricGroup and Permutations which use SymmetricGroupAlgebra. This ticket fixes this.

CC: @tscrim @darijgr @sagetrac-sage-combinat @avirmaux

Component: algebra

Author: Nicolas M. Thiéry

Branch/Commit: db95e00

Reviewer: Aladin Virmaux

Issue created by migration from https://trac.sagemath.org/ticket/18273
@nthiery nthiery added this to the sage-6.7 milestone Apr 22, 2015
@nthiery
Copy link
Contributor Author

nthiery commented Apr 22, 2015

Branch: u/nthiery/categories/symmetricgroup_algebra_category_option-18273

@nthiery
Copy link
Contributor Author

nthiery commented Apr 22, 2015

New commits:

db95e0018273: add category option to SymmetricGroup.algebra and Permutations.algebra for consistency

@nthiery
Copy link
Contributor Author

nthiery commented Apr 22, 2015

Commit: db95e00

@nthiery
Copy link
Contributor Author

nthiery commented Apr 22, 2015

comment:3

This should be a rather trivial review. Thanks in advance!

@avirmaux
Copy link
Mannequin

avirmaux mannequin commented Apr 22, 2015

comment:5

Looks good to me.

@nthiery
Copy link
Contributor Author

nthiery commented Apr 22, 2015

Changed reviewer from Darij Grinberg to Aladin Virmaux

@vbraun
Copy link
Member

vbraun commented Apr 23, 2015

Changed branch from u/nthiery/categories/symmetricgroup_algebra_category_option-18273 to db95e00

@vbraun vbraun closed this as completed in 9c013fb Apr 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: algebra p: major / 3 t: bug
Projects
None yet
Milestone
sage-6.7
Development

No branches or pull requests
2 participants
@nthiery @vbraun