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 coproduct, antipode and counit in k_dual.py #14237

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

Implements coproduct, antipode and counit in k_dual.py #14237

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

Comments

@zabrocki
Copy link
Mannequin

zabrocki mannequin commented Mar 6, 2013

The coalgebra and Hopf structure of the k-bounded quotient were not previously implemented. Ticket #14161 adds a test for the antipode in Hopf algebras which will require these methods.

Apply

  • trac_14237_hopf_kdual-mz.patch
  • trac_14237-hopf_kdual-review-ts.patch
  • trac_14237_change_two_words-mz.patch

Depends on #14228

CC: @anneschilling @saliola @sagetrac-chrisjamesberg

Component: combinatorics

Keywords: coproduct, symmetric functions, antipode

Author: Mike Zabrocki

Reviewer: Travis Scrimshaw

Merged: sage-5.10.beta5

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

@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

This ticket may have an impact on #13991. It will certainly add more tests to k_dual.py, but I hope changes to this file will improve performance rather than degrade it.

@zabrocki
Copy link
Mannequin Author

zabrocki mannequin commented Mar 6, 2013

Dependencies: #14228

@zabrocki
Copy link
Mannequin Author

zabrocki mannequin commented Mar 6, 2013

comment:2

I just noticed that #13991 closed and this was a duplicate with #14228. The changes that I make here will have to be made compatible.

@zabrocki
Copy link
Mannequin Author

zabrocki mannequin commented Mar 7, 2013

Attachment: trac_14237_hopf_kdual-mz.patch.gz

@zabrocki
Copy link
Mannequin Author

zabrocki mannequin commented Mar 7, 2013

comment:3

This version applies on #14228 and cleans up the k_dual.py file a bit.

@zabrocki zabrocki mannequin added the s: needs review label Mar 7, 2013
@tscrim
Copy link
Collaborator

tscrim commented May 19, 2013

Attachment: trac_14237-hopf_kdual-review-ts.patch.gz

@tscrim
Copy link
Collaborator

tscrim commented May 19, 2013

comment:4

Hey Mike,

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

Best,

Travis

@tscrim
Copy link
Collaborator

tscrim commented May 19, 2013

Reviewer: Travis Scrimshaw

@zabrocki

This comment has been minimized.

@zabrocki
Copy link
Mannequin Author

zabrocki mannequin commented May 20, 2013

comment:5

Attachment: trac_14237_change_two_words-mz.patch.gz

I completely forgot these tickets were here. I changed a couple of words because I thought the comment in the documentation that I wrote was unclear.

@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

2 participants