Yet another set of fixes:

http://hosted.smartbear.com/sympy/go?page=ReviewDisplay&reviewid=6

http://github.com/jegerjensen/sympy/compare/master...codegen_C4

Øyvind

On 2 Aug, 14:31, Øyvind Jensen <jensen.oyv...@gmail.com> wrote:
> I have uploaded new fixes here:
>
> http://hosted.smartbear.com/sympy/go?page=ReviewDisplay&reviewid=6
>
> If you prefer github, I'll read those comments too.  NOTE: the branch
> corresponding to the code on smartbear is called
> 'second_review_round':  
> http://github.com/jegerjensen/sympy/tree/second_review_round
>
> Øyvind
>
> On 28 Jul, 23:52, Øyvind Jensen <jensen.oyv...@gmail.com> wrote:
>
> > On 27 Jul, 19:07, Brian Granger <elliso...@gmail.com> wrote:
>
> > > Oyvind,
>
> > > I don't know if you saw the new operator priority stuff that I did this
> > > weekend.  This will make it possible to more reliably have Expr subclasses
> > > that define their own binary special methods (__mul__, __add__, etc._) 
> > > that
> > > are always called based on priority of the class.  This could be useful 
> > > when
> > > multiplying tensors, etc. to do compatibility testing.  Ondrej, pushed 
> > > this
> > > to sympy master in the last 24 hours.
>
> > I saw it, and thought it could be useful.  However, I can not
> > prioritize it right now. Maybe we should just create an issue and do
> > it later when things have stabilized.
>
> > Øyvind
>
> > > Cheers,
>
> > > Brian
>
> > > On Mon, Jul 26, 2010 at 1:36 PM, Øyvind Jensen 
> > > <jensen.oyv...@gmail.com>wrote:
>
> > > > I have uploaded my work on the C code printer, C code generator and the
> > > > tensor module to smartbear for review:
>
> > > >http://hosted.smartbear.com/sympy/go?page=ReviewDisplay&reviewid=6
>
> > > > The ability to create code that loops over array arguments is much
> > > > improved, mainly because the contraction logic is now handled in the
> > > > tensor module.  I added Andy as reviewer, but anyone can add themselves
> > > > to the review to be able to create comments and defects.
>
> > > > The code is also on github:
> > > >http://github.com/jegerjensen/sympy/tree/codegen_C2
>
> > > > Cheers,
> > > > Øyvind
>
> > > > --
> > > > 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