Opened 8 years ago

Closed 8 years ago

#14235 closed enhancement (fixed)

Implements antipode and counit in new_kschur.py

Reported by: zabrocki Owned by: Mike Zabrocki
Priority: minor Milestone: sage-5.10
Component: combinatorics Keywords: symmetric functions, antipode
Cc: aschilling, saliola, chrisjamesberg Merged in: sage-5.10.beta5
Authors: Mike Zabrocki Reviewers: Travis Scrimshaw
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #13743 Stopgaps:

Status badges

Description (last modified by zabrocki)

At t=1 the space where the k-Schur functions live is a Hopf algebra but the antipode and counit were never implemented because they were not required. Ticket #14161 adds a test for the antipode that requires a counit method and so doc tests in new_kschur.py will fail unless these methods are implemented.

Apply

  • trac_14235_hopf_kschur-mz.patch
  • trac_14235-hopf_kschur-review-ts.patch

Attachments (2)

trac_14235_hopf_kschur-mz.patch (5.8 KB) - added by zabrocki 8 years ago.
trac_14235-hopf_kschur-review-ts.patch (3.9 KB) - added by tscrim 8 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 8 years ago by zabrocki

Not sure whether to call this a 'defect' or an 'enhancement'. I am going with the latter.

I also am moving the coproduct method from the ElementMethods to the ParentMethods.

comment:2 Changed 8 years ago by zabrocki

  • Status changed from new to needs_review

comment:3 Changed 8 years ago by zabrocki

  • Dependencies set to #13743

This patch is now rebased against #13743

comment:4 Changed 8 years ago by chapoton

There is a typo here :

counit is the constant terms

should be

counit is the constant term

Changed 8 years ago by zabrocki

comment:5 Changed 8 years ago by zabrocki

I've corrected this typo.

Changed 8 years ago by tscrim

comment:6 Changed 8 years ago by tscrim

Hey Mike,

I've uploaded a review patch which makes some minor tweaks to the doc. If you're happy with the changes, you can set this to positive review.

Best,
Travis

comment:7 Changed 8 years ago by tscrim

  • Reviewers set to Travis Scrimshaw

comment:8 Changed 8 years ago by zabrocki

  • Status changed from needs_review to positive_review

Thanks Travis for coming back to both of these patches. They long fell off of my radar. -Mike

comment:9 Changed 8 years ago by zabrocki

  • Description modified (diff)

comment:10 Changed 8 years ago by jdemeyer

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