#9759: Addition of SI prefixes capabilities to the units module
-------------------------------+--------------------------------------------
       Reporter:  cousteau     |         Owner:  burcin    
           Type:  enhancement  |        Status:  needs_work
       Priority:  trivial      |     Milestone:  sage-5.1  
      Component:  symbolics    |    Resolution:            
       Keywords:  sd40.5       |   Work issues:            
Report Upstream:  N/A          |     Reviewers:            
        Authors:  Javier Mora  |     Merged in:            
   Dependencies:               |      Stopgaps:            
-------------------------------+--------------------------------------------
Changes (by kcrisman):

  * keywords:  => sd40.5
  * status:  needs_review => needs_work


Comment:

 I agree with Burcin that this is a bit hackish.
 > The truth is that this patch was created as a suggestion done in #sage-
 devel; it was done pretty fast and probably not elegantly. I later
 submitted another patch with a better implementation (see ticket #9778),
 which also added the basics for `LaTeX` representation of the units.
 So should this be closed in favor of #9778?

 Also,
 {{{
 def _add_si_property_(prefix):
             setattr(UnitExpression, prefix, property(lambda self:
 self._si_prefix_(prefix)))

 for prefix in unitdict['si_prefixes']:
      _add_si_property_(prefix)
 }}}
 seems to be missing a doctest in the underscore function.

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