#13089: Implement weighted projective spaces.
--------------------------------------+-------------------------------------
       Reporter:  davideklund         |         Owner:  AlexGhitza        
           Type:  enhancement         |        Status:  needs_work        
       Priority:  major               |     Milestone:  sage-5.3          
      Component:  algebraic geometry  |    Resolution:                    
       Keywords:                      |   Work issues:                    
Report Upstream:  N/A                 |     Reviewers:  Andrey Novoseltsev
        Authors:  David Eklund        |     Merged in:                    
   Dependencies:                      |      Stopgaps:                    
--------------------------------------+-------------------------------------
Changes (by novoselt):

  * status:  needs_review => needs_work
  * reviewer:  => Andrey Novoseltsev


Comment:

 Thanks for trying to use quotient lattices, I though I took care of
 length, but apparently not for all cases. Let's move on with WPs as is,
 just a couple little picks:

 Instead of
 {{{
 print "keyword argument %s (value %s) will be ignored" % (key,kw[key])
 }}}
 you should raise an exception, since ideally we want
 {{{
 def WP(self, *q, K=QQ, names="z+"):
 }}}
 and it would. (Showing the traceback that lead to it instead of just the
 error.)

 It is also deprecated in Python to use `raise Error, "message"`, rather it
 should be `raise Error("message")`.

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