#4352: [with patch, needs work] add support for weight vectors to
gran/groebner_fan
------------------------------------------------+---------------------------
 Reporter:  mhampton                            |        Owner:  mhampton  
     Type:  enhancement                         |       Status:  new       
 Priority:  minor                               |    Milestone:  sage-3.2.2
Component:  interfaces                          |   Resolution:            
 Keywords:  gfan, groebner_fan, weight vectors  |  
------------------------------------------------+---------------------------
Changes (by was):

  * summary:  [with patch, needs review] add support for weight vectors to
              gran/groebner_fan => [with patch, needs work]
              add support for weight vectors to
              gran/groebner_fan
  * milestone:  sage-feature => sage-3.2.2

Comment:

 REVIEW:

 patch applies and passes test.  Code raises major red flag!!
 {{{
                 ans =
 eval(ans.replace('{','').replace('}','').replace('\n',''))
 }}}

 If the output -- which you make into vectors over QQ ever actually has any
 rational numbers, then eval will do very bad things to them, e.g.,
 {{{
 sage: eval('2/3')
 0
 }}}
 Oops!

 Use sage_eval instead:
 {{{
 sage: sage_eval('2/3')
 2/3
 }}}

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4352#comment:2>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of 
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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