#10683: Hermitian inner product
-----------------------------------+----------------------------------------
   Reporter:  rbeezer              |       Owner:  jason, was  
       Type:  enhancement          |      Status:  needs_review
   Priority:  minor                |   Milestone:  sage-4.7    
  Component:  linear algebra       |    Keywords:              
     Author:  Rob Beezer           |    Upstream:  N/A         
   Reviewer:  Karl-Dieter Crisman  |      Merged:              
Work_issues:                       |  
-----------------------------------+----------------------------------------
Changes (by rbeezer):

  * status:  needs_info => needs_review


Comment:

 Replying to [comment:5 kcrisman]:
 > So I don't know whether this is a reason to deny positive review.  What
 do you think?  Separate ticket, or fix here?

 This routine is an extension of the idea of a dot product.  So I'd say the
 dot product is broken and needs to be more careful about mixing the two
 base rings.  Maybe it should check compatibility, and if there is no
 conflict, then go on its way.  If there is a conflict, the vectors should
 be converted to a common base ring, then do the dot product.  If that does
 not work, then the routine should fail gracefully.

 So I'd like that to be on another ticket.  If you make the ticket and cc
 me, then I'll put it on my list for SD 29 next week, and coordinate with
 this ticket, if necessary.  But I'd like to see this go in pretty much as-
 is and let improvements to the "plain" dot product improve this routine
 from outside.

 Thanks,
 Rob

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10683#comment:6>
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.

Reply via email to