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

symmetric and exterior powers of characters #10175

Closed
vbraun opened this issue Oct 26, 2010 · 10 comments
Closed

symmetric and exterior powers of characters #10175

vbraun opened this issue Oct 26, 2010 · 10 comments

Comments

@vbraun
Copy link
Member

vbraun commented Oct 26, 2010

The gap-functions for computing exterior and symmetric powers of characters are not exposed. This patch implements these.

Apply attachment: trac_10175_anti_symmetric_product_characters.patch

Component: group theory

Author: Volker Braun

Reviewer: David Roe

Merged: sage-5.0.beta3

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

@vbraun
Copy link
Member Author

vbraun commented Dec 15, 2011

Updated patch

@vbraun
Copy link
Member Author

vbraun commented Dec 15, 2011

comment:2

Attachment: trac_10175_anti_symmetric_product_characters.2.patch.gz

Rediffed for sage-4.8.alpha4

@vbraun

This comment has been minimized.

@roed314
Copy link
Contributor

roed314 commented Jan 5, 2012

comment:3

Just a couple comments:

  • You need to raise the NotImplementedError on line 281, and there are a couple other points in the file where this same mistake is made.

  • In the documentation of __mul__, you only allow multiplication by an integer. But you can in fact multiply by other scalars that coerce into Gap: number field elements for example.

Other than that, I'll run the tests once I build a new copy of Sage on sage.math and fly to Boston.

@roed314
Copy link
Contributor

roed314 commented Jan 5, 2012

Reviewer: David Roe

@vbraun
Copy link
Member Author

vbraun commented Jan 8, 2012

Attachment: trac_10175_anti_symmetric_product_characters.patch.gz

Updated patch

@vbraun
Copy link
Member Author

vbraun commented Jan 8, 2012

comment:4

Fixed in the updated patch. Also, made sage -coverage happy.

@roed314
Copy link
Contributor

roed314 commented Feb 2, 2012

comment:5

Looks good to me. Sorry for the delay.

@jdemeyer

This comment has been minimized.

@jdemeyer
Copy link

jdemeyer commented Feb 6, 2012

Merged: sage-5.0.beta3

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

4 participants