On Fri, Apr 2, 2010 at 3:00 PM, Mateusz Paprocki <[email protected]> wrote: > Hi, > > On Fri, Mar 19, 2010 at 08:11:21PM -0700, Ondrej Certik wrote: >> On Fri, Mar 19, 2010 at 7:54 PM, Ted Horst <[email protected]> wrote: >> > >> > On 2010-03-17, at 21:28, Ondrej Certik wrote: >> > >> >> Hi, >> >> >> >> we just fixed the last bugs in the Mateusz's polys branch and pushed >> >> it in our main git repo. All tests pass for all kinds of >> >> configurations that I tried, but if possible, please test it (I didn't >> >> test on Mac nor Windows). >> >> >> >> Just to get some idea, Mateusz did about 150 patches and it implements >> >> all kinds of polynomial algorithms, see e.g. this patch for some >> >> details: >> >> >> >> >> >> http://git.sympy.org/?p=sympy.git;a=commit;h=31c9aa43ffc7bc031493fc4ea27b74dc50c4d035 >> >> >> >> It should fix all kinds of bugs and also it's quite fast and robust. >> >> E.g. this failed: >> >> >> >> In [1]: f = expand((x+y+z)**20) >> >> >> >> In [2]: factor(f) >> >> Out[2]: >> >> 20 >> >> (x + y + z) >> >> >> >> and now it works. >> >> >> >> Plus the core polynomials are cythonized in pure Cython mode (so far >> >> only about 2x speedup), but I think we'll figure out ways to speed it >> >> up more using Cython. >> >> >> >> Ondrej >> >> >> > >> > I was excited to see the first cython bits in the main repo, but I am >> > having >> > a few problems. >> > >> > I'm on a ppc mac running 10.5.8 and using the system python 2.5.1 and the >> > latest head of cython (12507ec6fa91) from >> > http://hg.cython.org/cython-devel. >> > >> > First, it looks like lambda is not supported by this version of cython >> > (error below). Are you using the cython-closure branch? >> > >> > Second, it looks like the cythonized decorator is not having the desired >> > effect. I'm guessing this is because the cython.locals decorator has to >> > actually be seen by cython (the decorator itself doesn't do anything). >> > Could this be done using a pxd file? >> >> I just tried the same Cython head and I am getting the exact same >> cython error as you do. >> >> Mateusz, how should we get this working? Last time I tried it was at >> the scipy conference and it worked, but it was just some prototype. >> > > this was like half a year ago ;) I fixed this issue in polys8 branch > (http://github.com/mattpap/sympy-polys/commits/polys8). The problem > was mainly with lambda, which still isn't supported, and with > decomposition of tuples in arguments to functions.
Indeed, that fixes it for me. Can I push your branch in? Ondrej -- You received this message because you are subscribed to the Google Groups "sympy" 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/sympy?hl=en.
