Opened 12 years ago

Closed 10 years ago

#10175 closed enhancement (fixed)

symmetric and exterior powers of characters

Reported by: vbraun Owned by: joyner
Priority: major Milestone: sage-5.0
Component: group theory Keywords:
Cc: Merged in: sage-5.0.beta3
Authors: Volker Braun Reviewers: David Roe
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by jdemeyer)

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

Apply trac_10175_anti_symmetric_product_characters.patch

Attachments (2)

trac_10175_anti_symmetric_product_characters.2.patch (19.0 KB) - added by vbraun 10 years ago.
Updated patch
trac_10175_anti_symmetric_product_characters.patch (20.7 KB) - added by vbraun 10 years ago.
Updated patch

Download all attachments as: .zip

Change History (9)

comment:1 Changed 12 years ago by vbraun

  • Status changed from new to needs_review

Changed 10 years ago by vbraun

Updated patch

comment:2 Changed 10 years ago by vbraun

  • Description modified (diff)

Rediffed for sage-4.8.alpha4

comment:3 Changed 10 years ago by roed

  • Reviewers set to David Roe
  • Status changed from needs_review to needs_work

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.

Changed 10 years ago by vbraun

Updated patch

comment:4 Changed 10 years ago by vbraun

  • Status changed from needs_work to needs_review

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

comment:5 Changed 10 years ago by roed

  • Status changed from needs_review to positive_review

Looks good to me. Sorry for the delay.

comment:6 Changed 10 years ago by jdemeyer

  • Description modified (diff)

comment:7 Changed 10 years ago by jdemeyer

  • Merged in set to sage-5.0.beta3
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.