Ondrej,

Thanks!  This will be a huge help to us.

Cheers,

Brian

On Tue, Jul 27, 2010 at 12:10 AM, Ondrej Certik <ond...@certik.cz> wrote:

> On Sun, Jul 25, 2010 at 12:34 PM, Brian Granger <elliso...@gmail.com>
> wrote:
> > Hi,
> > Here is the final version of my binary operator priority branch:
> > http://github.com/ellisonbg/sympy/tree/priority
> > To address comments I have:
> > * Created a decorator @call_highest_priority that is in
> > sympy.core.decorators to encapsulate the logic.
> > * Added documentation and tests.
> > * On my system the test suite slows down only 1-2% with this patch.  It
> is
> > important to remember that this patch will allow the creation of custom
> > Mul/Add/Pow classes for things like non-commutative entities.  In the
> > long run, this will simplify the existing Mul/Add/Pow classes and speed
> > things up.
> > I would like to merge this soon if possible.
>
> I have tested this and the implementation is simple. If it turns out
> that this brings some super difficult problems, it can be easily
> reverted (just the decorator has to be commented out).
>
> All tests pass for me, the speed is acceptable for what I have seen,
> so I have pushed this in.
>
> Ondrej
>
> --
> You received this message because you are subscribed to the Google Groups
> "sympy-patches" group.
> To post to this group, send email to sympy-patc...@googlegroups.com.
> To unsubscribe from this group, send email to
> sympy-patches+unsubscr...@googlegroups.com<sympy-patches%2bunsubscr...@googlegroups.com>
> .
> For more options, visit this group at
> http://groups.google.com/group/sympy-patches?hl=en.
>
>


-- 
Brian E. Granger, Ph.D.
Assistant Professor of Physics
Cal Poly State University, San Luis Obispo
bgran...@calpoly.edu
elliso...@gmail.com

-- 
You received this message because you are subscribed to the Google Groups 
"sympy-patches" group.
To post to this group, send email to sympy-patc...@googlegroups.com.
To unsubscribe from this group, send email to 
sympy-patches+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/sympy-patches?hl=en.

Reply via email to