#8620: Rogue minus sign in sage.modular.modsym.ambient.diamond_bracket_operator
-----------------------------+----------------------------------------------
Reporter: davidloeffler | Owner: craigcitro
Type: defect | Status: needs_review
Priority: major | Milestone: sage-4.4
Component: modular forms | Keywords: modular symbols
Author: | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
-----------------------------+----------------------------------------------
Changes (by davidloeffler):
* status: new => needs_review
Old description:
> There is a minus sign in the code for diamond operators which shouldn't
> be there; what actually gets returned is the diamond operator times the
> star involution! In particular, ```< 1 >``` really ought to be the
> identity map. This patch corrects the error and adds a doctest to prove
> it. (This is needed for some code I wrote with Jared Weinstein at the
> 2010 Montreal conference, in which it's really vital to work with sign 0
> symbols.)
New description:
There is a minus sign in the code for diamond operators which shouldn't be
there; what actually gets returned is the diamond operator times the star
involution! In particular, ```<1>``` really ought to be the identity map.
This patch corrects the error and adds a doctest to prove it. (This is
needed for some code I wrote with Jared Weinstein at the 2010 Montreal
conference, in which it's really vital to work with sign 0 symbols.)
--
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8620#comment:1>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
