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

add counit method to QSym #14163

Closed
saliola opened this issue Feb 22, 2013 · 10 comments
Closed

add counit method to QSym #14163

saliola opened this issue Feb 22, 2013 · 10 comments

Comments

@saliola
Copy link

saliola commented Feb 22, 2013

The Hopf algebra of quasisymmetric functions is missing the counit method.

sage: QSym = QuasiSymmetricFunctions(QQ)
sage: F = QSym.Fundamental()
sage: F.counit
sage:

Without the counit, _test_antipode at #14161 fails for QSym

Apply: trac_14163_add_counit_qsym-fs.patch

CC: @zabrocki

Component: combinatorics

Keywords: qsym, hopf algebras

Author: Mike Zabrocki, Franco Saliola

Reviewer: Franco Saliola, Mike Zabrocki

Merged: sage-5.8.beta1

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

@zabrocki
Copy link
Mannequin

zabrocki mannequin commented Feb 22, 2013

Author: Mike Zabrocki

@zabrocki
Copy link
Mannequin

zabrocki mannequin commented Feb 22, 2013

comment:1

Added counit_on_basis to the ParentMethods of the Bases class in QSym. This is a copy of what is in NSym.

@zabrocki zabrocki mannequin added the s: needs review label Feb 22, 2013
@saliola
Copy link
Author

saliola commented Feb 22, 2013

apply only this patch

@saliola

This comment has been minimized.

@saliola
Copy link
Author

saliola commented Feb 22, 2013

comment:2

Attachment: trac_14163_add_counit_qsym-fs.patch.gz

Since the code is identical to that of NSym, I put the method in ParentMethods for bases of both algebras. Attached it an updated patch. Mike, if you like the change, then set this to positive review.

@saliola
Copy link
Author

saliola commented Feb 22, 2013

Reviewer: Franco Saliola, Mike Zabrocki

@saliola
Copy link
Author

saliola commented Feb 22, 2013

Changed author from Mike Zabrocki to Mike Zabrocki, Franco Saliola

@nthiery
Copy link
Contributor

nthiery commented Feb 22, 2013

comment:3

This really should be added to GradedConnectedHopfAlgebrasWithBasis. But we don't have this category yet (it will be easy when the functorial construction patch will be there). So good for now, but please open a ticket to remind us to create this category and put the counit there!

Thanks!

@jdemeyer
Copy link

Changed work issues from implement the counit to none

@jdemeyer
Copy link

Merged: sage-5.8.beta1

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