Lambdify namespace order bugfix

Hi, I posted to report a problem with lambdify and namespaces to the sympy mailing list a while ago: http://groups.google.com/group/sympy/browse_thread/thread/bb2c16413c49b6f9?fwc=2 but was stirred by the silence to go and find and fix the bug (patch attached, with explanatory test). Sorry not

Re: Lambdify namespace order bugfix

Hi Matthew! On Thu, Jul 1, 2010 at 1:53 PM, Matthew Brett wrote: > Hi, > > I posted to report a problem with lambdify and namespaces to the sympy > mailing list a while ago: > > http://groups.google.com/group/sympy/browse_thread/thread/bb2c16413c49b6f9?fwc=2 > > but was stirred by the silence to

Re: Lambdify namespace order bugfix

For some reason, this patch removed the ">>> from sympy import lambdify" line from the docstring, which now causes doctest failures. Aaron Meurer On Jul 2, 2010, at 6:05 PM, Ondrej Certik wrote: > Hi Matthew! > > On Thu, Jul 1, 2010 at 1:53 PM, Matthew Brett wrote: >> Hi, >> >> I posted to r

Re: Lambdify namespace order bugfix

Oops, you are right. I just fixed that. I forgot to run doctests, and the reason is that I got a failure in mpmath tests, some vizualization problem, when I run it remotely. ond...@raven:~/repos/sympy(master)$ bin/test sympy/mpmath/tests/test_visualization.py = test pr

Re: Lambdify namespace order bugfix

> Oops, you are right. > > I just fixed that. I forgot to run doctests, and the reason is that I > got a failure in mpmath tests, some vizualization problem, when I run > it remotely. See issue 1956. Aaron Meurer -- You received this message because you are subscribed to the Google Groups "sy

Re: Lambdify namespace order bugfix

On Sun, Jul 4, 2010 at 10:45 AM, Aaron S. Meurer wrote: >> Oops, you are right. >> >> I just fixed that. I forgot to run doctests, and the reason is that I >> got a failure in mpmath tests, some vizualization problem, when I run >> it remotely. > > See issue 1956. I just fixed that too. Ondrej

Re: Lambdify namespace order bugfix

On 4 Jul, 19:54, Ondrej Certik wrote: > On Sun, Jul 4, 2010 at 10:45 AM, Aaron S. Meurer wrote: > > >> Oops, you are right. > > >> I just fixed that. I forgot to run doctests, and the reason is that I > >> got a failure in mpmath tests, some vizualization problem, when I run > >> it remotely. >

Re: Lambdify namespace order bugfix

Hi, > The patch is +1, and I've pushed it in. It's in this repo: > > http://github.com/sympy/sympy Great - thanks - I'd actually already switched to github.com/sympy/sympy > Btw, I just talked to Andy in Pilsen, how he got involved with sympy > and he told me the exact same experience as you did

Re: Lambdify namespace order bugfix

On Mon, Jul 5, 2010 at 5:16 AM, Øyvind Jensen wrote: > > > On 4 Jul, 19:54, Ondrej Certik wrote: >> On Sun, Jul 4, 2010 at 10:45 AM, Aaron S. Meurer wrote: >> >> >> Oops, you are right. >> >> >> I just fixed that. I forgot to run doctests, and the reason is that I >> >> got a failure in mpmath t

Re: Lambdify namespace order bugfix

ma., 05.07.2010 kl. 10.08 -0700, skrev Ondrej Certik: > On Mon, Jul 5, 2010 at 5:16 AM, Øyvind Jensen wrote: > > > > > > On 4 Jul, 19:54, Ondrej Certik wrote: > >> On Sun, Jul 4, 2010 at 10:45 AM, Aaron S. Meurer > >> wrote: > >> > >> >> Oops, you are right. > >> > >> >> I just fixed that. I fo