Oh, and if this version works better in Sage than the previous one, it's because we actually ran the Sage tests before releasing this time :) (there were a ton of errors, many existing before the previous release or two).
Aaron Meurer On Thu, Jun 30, 2011 at 4:44 PM, Aaron Meurer <[email protected]> wrote: > I assume you mean doctests in sage. If you're referring to a doctest > in SymPy, please point it out to us, as to our knowledge there are > none in the release. > > And let us know if it's necessary to patch SymPy, so we can fix it upstream. > > Oh, and I happen to know that Sage uses symbols(each_char=False) > somewhere in the code. each_char is now False by default, and there > is no need to set it to that. We originally removed this option, but > added it back in as a deprecated feature when I noticed that Sage uses > it. If you want each_char=True, you should replace it with separating > symbols by spaces or commas, like symbols('x y z'). And if you have > any bare symbols('xyz') in the code, with multiple characters and no > each_char, you will need to change that. Also, if you run with python > -Wd, you will see a DeprecationWarning for this. > > Aaron Meurer > > On Thu, Jun 30, 2011 at 4:12 PM, François > <[email protected]> wrote: >> I had a quick look at it. I think it goes down better than sympy-0.6.7, >> there are a few minor doctests to fix (some answers are now x+2 instead of >> 2+x) but some code in sage needs adjusting before we ship: >> File "/usr/share/sage/devel/sage-main/sage/calculus/test_sympy.py", line >> 294, in __main__.example_0 >> Failed example: >> f = e.series(x, Integer(0), Integer(10)); f###line 111:_sage_ >>> f = >> e.series(x, 0, 10); f >> Exception raised: >> Traceback (most recent call last): >> File "/usr/bin/ncadoctest.py", line 1231, in run_one_test >> self.run_one_example(test, example, filename, compileflags) >> File "/usr/bin/sagedoctest.py", line 38, in run_one_example >> OrigDocTestRunner.run_one_example(self, test, example, filename, >> compileflags) >> File "/usr/bin/ncadoctest.py", line 1172, in run_one_example >> compileflags, 1) in test.globs >> File "<doctest __main__.example_0[43]>", line 1, in <module> >> f = e.series(x, Integer(0), Integer(10)); f###line 111:_sage_ >>> >> f = e.series(x, 0, 10); f >> File "/usr/lib64/python2.7/site-packages/sympy/core/expr.py", line >> 1517, in series >> if x0 in [S.Infinity, S.NegativeInfinity]: >> File "integer.pyx", line 850, in >> sage.rings.integer.Integer.__richcmp__ (sage/rings/integer.c:7498) >> File "element.pyx", line 832, in >> sage.structure.element.Element._richcmp (sage/structure/element.c:6598) >> File "coerce.pyx", line 889, in >> sage.structure.coerce.CoercionModel_cache_maps.canonical_coercion >> (sage/structure/coerce.c:8193) >> File "/usr/lib64/python2.7/site-packages/sympy/core/numbers.py", line >> 892, in _sage_ >> return sage.Integer(self.p)/sage.Integer(self.q) >> File "element.pyx", line 1555, in >> sage.structure.element.RingElement.__div__ (sage/structure/element.c:12011) >> File "integer.pyx", line 1662, in sage.rings.integer.Integer._div_ >> (sage/rings/integer.c:11470) >> File "integer_ring.pyx", line 286, in >> sage.rings.integer_ring.IntegerRing_class._div >> (sage/rings/integer_ring.c:5167) >> ZeroDivisionError: Rational division by zero >> That's the only major problem I detected. >> Francois >> >> -- >> You received this message because you are subscribed to the Google Groups >> "sympy" group. >> To view this discussion on the web visit >> https://groups.google.com/d/msg/sympy/-/G0dzmKE2fBQJ. >> 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. >> > -- 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.
