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

Implements antipode and counit in new_kschur.py #14235

Closed
zabrocki mannequin opened this issue Mar 6, 2013 · 10 comments
Closed

Implements antipode and counit in new_kschur.py #14235

zabrocki mannequin opened this issue Mar 6, 2013 · 10 comments

Comments

@zabrocki
Copy link
Mannequin

zabrocki mannequin commented Mar 6, 2013

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

Depends on #13743

CC: @anneschilling @saliola @sagetrac-chrisjamesberg

Component: combinatorics

Keywords: symmetric functions, antipode

Author: Mike Zabrocki

Reviewer: Travis Scrimshaw

Merged: sage-5.10.beta5

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

@zabrocki zabrocki mannequin added this to the sage-5.10 milestone Mar 6, 2013
@zabrocki zabrocki mannequin added c: combinatorics labels Mar 6, 2013
@zabrocki
Copy link
Mannequin Author

zabrocki mannequin commented Mar 6, 2013

comment:1

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.

@zabrocki zabrocki mannequin added the s: needs review label Mar 6, 2013
@zabrocki
Copy link
Mannequin Author

zabrocki mannequin commented Mar 11, 2013

comment:3

This patch is now rebased against #13743

@zabrocki
Copy link
Mannequin Author

zabrocki mannequin commented Mar 11, 2013

Dependencies: #13743

@fchapoton
Copy link
Contributor

comment:4

There is a typo here :

counit is the constant terms

should be

counit is the constant term

@zabrocki
Copy link
Mannequin Author

zabrocki mannequin commented Mar 17, 2013

comment:5

Attachment: trac_14235_hopf_kschur-mz.patch.gz

I've corrected this typo.

@tscrim
Copy link
Collaborator

tscrim commented May 19, 2013

comment:6

Attachment: trac_14235-hopf_kschur-review-ts.patch.gz

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

@tscrim
Copy link
Collaborator

tscrim commented May 19, 2013

Reviewer: Travis Scrimshaw

@zabrocki
Copy link
Mannequin Author

zabrocki mannequin commented May 20, 2013

comment:8

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

@zabrocki

This comment has been minimized.

@jdemeyer
Copy link

Merged: sage-5.10.beta5

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

3 participants