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: |
Description (last modified by )
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)
Change History (12)
comment:1 Changed 8 years ago by
comment:2 Changed 8 years ago by
- Status changed from new to needs_review
comment:3 Changed 8 years ago by
- Dependencies set to #13743
This patch is now rebased against #13743
comment:4 Changed 8 years ago by
There is a typo here :
counit is the constant terms
should be
counit is the constant term
Changed 8 years ago by
comment:5 Changed 8 years ago by
I've corrected this typo.
Changed 8 years ago by
comment:6 Changed 8 years ago by
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
- Reviewers set to Travis Scrimshaw
comment:8 Changed 8 years ago by
- 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
- Description modified (diff)
comment:10 Changed 8 years ago by
- Merged in set to sage-5.10.beta5
- Resolution set to fixed
- Status changed from positive_review to closed
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 theElementMethods
to theParentMethods
.