Doctest interfaces/psage.py #12061

ppurka · 2011-11-20T08:33:00Z

Towards completing #12024.

Apply only attachment: trac_12061.txt to devel/sage.

Component: interfaces

Work Issues: find out why doctests change every sage version

Author: Punarbasu Purkayastha

Issue created by migration from https://trac.sagemath.org/ticket/12061

The text was updated successfully, but these errors were encountered:

ppurka · 2011-11-20T09:02:18Z

Apply to devel/sage

ppurka · 2011-11-20T09:07:13Z

comment:1

Attachment: trac_12061-doctest_interfaces_psage.patch.gz

Form some reason doctesting this file goes into an infinite recursion with twisted, but all the commands work on the Sage command line. I had to run it like this to see the output .sage/tmp/psage_26566.py is the file Sage created while trying to doctest.

PYTHONVERBOSE=1 ~/Installations/sage-4.7.2/sage -python psage_26566.py 2>&1 | tee ~/test.out

And ~/test.out contains in particular this output just before the infinite recursion starts:

import sage.modules.vector_double_dense # dynamically loaded from /home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/sage/modules/vector_double_dense.so
dlopen("/home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/sage/modules/vector_real_double_dense.so", 2);
import sage.modules.vector_real_double_dense # dynamically loaded from /home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/sage/modules/vector_real_double_dense.so
import sage.finance.time_series # dynamically loaded from /home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/sage/finance/time_series.so
import sage.stats.intlist # dynamically loaded from /home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/sage/stats/intlist.so
# /home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/sage/plot/plot3d/implicit_plot3d.pyc matches /home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/sage/plot/plot3d/implicit_plot3d.py
import sage.plot.plot3d.implicit_plot3d # precompiled from /home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/sage/plot/plot3d/implicit_plot3d.pyc
dlopen("/home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/sage/plot/plot3d/implicit_surface.so", 2);
import sage.plot.plot3d.implicit_surface # dynamically loaded from /home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/sage/plot/plot3d/implicit_surface.so
dlopen("/home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/sage/plot/complex_plot.so", 2);
import sage.plot.complex_plot # dynamically loaded from /home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/sage/plot/complex_plot.so
Exception IOError: IOError(2, 'No such file or directory') in <bound method SageElement.__del__ of <<currently executing code>>> ignored
dlopen("/home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/sage/libs/flint/flint.so", 2);
import sage.libs.flint.flint # dynamically loaded from /home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/sage/libs/flint/flint.so
# /home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/Twisted-9.0.0-py2.6-linux-x86_64.egg/twisted/internet/reactor.pyc matches /home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/Twisted-9.0.0-py2.6-linux-x86_64.egg/twisted/internet/reactor.py
import twisted.internet.reactor # precompiled from /home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/Twisted-9.0.0-py2.6-linux-x86_64.egg/twisted/internet/reactor.pyc
# /home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/Twisted-9.0.0-py2.6-linux-x86_64.egg/twisted/internet/selectreactor.pyc matches /home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/Twisted-9.0.0-py2.6-linux-x86_64.egg/twisted/internet/selectreactor.py
import twisted.internet.selectreactor # precompiled from /home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/Twisted-9.0.0-py2.6-linux-x86_64.egg/twisted/internet/selectreactor.pyc
# /home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/Twisted-9.0.0-py2.6-linux-x86_64.egg/twisted/internet/interfaces.pyc matches /home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/Twisted-9.0.0-py2.6-linux-x86_64.egg/twisted/internet/interfaces.py
import twisted.internet.interfaces # precompiled from /home/punarbasu/Installations/sage-4.7.2/local/lib/python2.6/site-packages/Twisted-9.0.0-py2.6-linux-x86_64.egg/twisted/internet/interfaces.pyc
Exception RuntimeError: 'maximum recursion depth exceeded in __subclasscheck__' in <type 'exceptions.RuntimeError'> ignored
Exception RuntimeError: RuntimeError('maximum recursion depth exceeded while calling a Python object',) in Exception RuntimeError: 'maximum recursion depth exceeded in __subclasscheck__' in <type 'exceptions.RuntimeError'> ignored
 ignored
Exception RuntimeError: 'maximum recursion depth exceeded in __subclasscheck__' in <type 'exceptions.RuntimeError'> ignored
Exception RuntimeError: RuntimeError('maximum recursion depth exceeded while calling a Python object',) in Exception RuntimeError: 'maximum recursion depth exceeded in __subclasscheck__' in <type 'exceptions.RuntimeError'> ignored
 ignored
Exception RuntimeError: 'maximum recursion depth exceeded in __subclasscheck__' in <type 'exceptions.RuntimeError'> ignored
Exception RuntimeError: RuntimeError('maximum recursion depth exceeded while calling a Python object',) in Exception RuntimeError: 'maximum recursion depth exceeded in __subclasscheck__' in <type 'exceptions.RuntimeError'> ignored
 ignored
Exception RuntimeError: 'maximum recursion depth exceeded in __subclasscheck__' in <type 'exceptions.RuntimeError'> ignored
Exception RuntimeError: RuntimeError('maximum recursion depth exceeded while calling a Python object',) in Exception RuntimeError: 'maximum recursion depth exceeded in __subclasscheck__' in <type 'exceptions.RuntimeError'> ignored
 ignored
Exception RuntimeError: 'maximum recursion depth exceeded in __subclasscheck__' in <type 'exceptions.RuntimeError'> ignored
Exception RuntimeError: RuntimeError('maximum recursion depth exceeded while calling a Python object',) in Exception RuntimeError: 'maximum recursion depth exceeded in __subclasscheck__' in <type 'exceptions.RuntimeError'> ignored
 ignored
Exception RuntimeError: 'maximum recursion depth exceeded in __subclasscheck__' in <type 'exceptions.RuntimeError'> ignored
Exception RuntimeError: RuntimeError('maximum recursion depth exceeded while calling a Python object',) in Exception RuntimeError: 'maximum recursion depth exceeded in __subclasscheck__' in <type 'exceptions.RuntimeError'> ignored
 ignored

ppurka · 2012-03-01T11:28:06Z

Attachment: trac_12061-doctest_interfaces_psage.2.patch.gz

Apply to devel/sage

ppurka · 2012-03-01T11:30:11Z

comment:2

Finally I have been able to figure out the cause of the infinite recursion. The fix is present in the patch.

ppurka · 2012-03-01T11:40:27Z

comment:3

Two functions are not directly or indirectly doctested: PSage._send_nowait() and PSageElement.is_locked(). Is the latter class even used??

jdemeyer · 2012-04-02T09:28:30Z

comment:4

Why add time.sleep(10)?

If you do, you also need to add "# long time".

ppurka · 2012-04-02T12:26:39Z

comment:5

Replying to @jdemeyer:

Why add time.sleep(10)?

If you do, you also need to add "# long time".

I did this after testing many different inputs and this was one input which doesn't take too long but which also doesn't give the result immediately. So, time.sleep(10) is there to wait until the computations are all over.

Is there a better way to test this? Otherwise, I will add # long time as you suggested.

ppurka · 2012-04-29T15:19:06Z

comment:6

Hmm.. I have a patch ready but I am unable to upload it. Something is wrong with trac. The upload page is simply stuck and never finishes the upload. I will just record the terminal output here and upload the patch some other day.

...2.experimental/devel/sage» ../../sage -t -long sage/interfaces/psage.py 
sage -t -long "devel/sage-main/sage/interfaces/psage.py"    
	 [24.8 s]
 
----------------------------------------------------------------------
All tests passed!
Total time for all tests: 24.8 seconds
...2.experimental/devel/sage» ../../sage -coverage sage/interfaces/psage.py 
----------------------------------------------------------------------
sage/interfaces/psage.py
ERROR: Please add a `TestSuite(s).run()` doctest.
SCORE sage/interfaces/psage.py: 71% (10 of 14)

Missing documentation:
	 * _start(self):
	 * _send_nowait(self, x):
	 * _object_class(self):
	 * is_locked(self):

----------------------------------------------------------------------

ppurka · 2012-05-01T08:46:56Z

Attachment: trac_12061.txt

Apply only this patch to devel/sage

ppurka · 2012-05-01T09:00:50Z

comment:7

There seems to be some weird problem with trac. I am unable to upload the original file since it contained

:trac:`1991`

instead of

:trac:`#1991`

Even after I did that change, I am unable to upload it as a .patch file. So I renamed the file and it works.

ppurka · 2012-05-26T14:51:01Z

Changed keywords from none to doctest sd40.5

ppurka · 2012-05-26T14:51:01Z

comment:8

Attachment: test.patch.gz

ppurka · 2012-05-27T10:09:43Z

comment:9

patchbot: apply trac_12061.txt

ppurka · 2012-05-27T10:23:12Z

comment:10

removing sd40.5. This PSage has extremely weird behavior which changes between several different versions of Sage. The doctests used to all pass earlier and now they don't.

ppurka · 2012-05-27T10:23:12Z

Changed keywords from doctest sd40.5 to none

jdemeyer · 2012-07-27T20:41:50Z

comment:11

Please fill in your real name as Author.

ppurka · 2012-07-28T12:52:28Z

comment:12

Sorry jdemeyer. It seems I tend to not fill up author fields. Will try to remember to do so from next time.

ppurka · 2012-07-28T12:52:28Z

Author: Punarbasu Purkayastha

ppurka · 2012-07-28T12:52:50Z

Work Issues: find out why doctests change every sage version

ppurka added this to the sage-5.11 milestone Nov 20, 2011

ppurka added c: interfaces labels Nov 20, 2011

ppurka assigned williamstein Nov 20, 2011

This comment has been minimized.

Sign in to view

ppurka added s: needs review and removed t: bug labels Mar 1, 2012

jdemeyer added s: needs work and removed s: needs review labels Apr 2, 2012

This comment has been minimized.

Sign in to view

ppurka added s: needs review and removed s: needs work labels May 1, 2012

ppurka added s: needs work and removed s: needs review labels Jul 28, 2012

jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013

sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014

sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014

sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014

mkoeppe removed this from the sage-6.4 milestone Dec 29, 2022

williamstein mentioned this issue Feb 22, 2013

90% doctest coverage thrust metaticket #12024

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Doctest interfaces/psage.py #12061

Doctest interfaces/psage.py #12061

ppurka commented Nov 20, 2011

ppurka commented Nov 20, 2011

ppurka commented Nov 20, 2011

ppurka commented Mar 1, 2012

ppurka commented Mar 1, 2012

This comment has been minimized.

ppurka commented Mar 1, 2012

jdemeyer commented Apr 2, 2012

ppurka commented Apr 2, 2012

ppurka commented Apr 29, 2012

ppurka commented May 1, 2012

ppurka commented May 1, 2012

This comment has been minimized.

ppurka commented May 26, 2012

ppurka commented May 26, 2012

ppurka commented May 27, 2012

ppurka commented May 27, 2012

ppurka commented May 27, 2012

jdemeyer commented Jul 27, 2012

ppurka commented Jul 28, 2012

ppurka commented Jul 28, 2012

ppurka commented Jul 28, 2012

Doctest interfaces/psage.py #12061

Doctest interfaces/psage.py #12061

Comments

ppurka commented Nov 20, 2011

ppurka commented Nov 20, 2011

ppurka commented Nov 20, 2011

ppurka commented Mar 1, 2012

ppurka commented Mar 1, 2012

This comment has been minimized.

ppurka commented Mar 1, 2012

jdemeyer commented Apr 2, 2012

ppurka commented Apr 2, 2012

ppurka commented Apr 29, 2012

ppurka commented May 1, 2012

ppurka commented May 1, 2012

This comment has been minimized.

ppurka commented May 26, 2012

ppurka commented May 26, 2012

ppurka commented May 27, 2012

ppurka commented May 27, 2012

ppurka commented May 27, 2012

jdemeyer commented Jul 27, 2012

ppurka commented Jul 28, 2012

ppurka commented Jul 28, 2012

ppurka commented Jul 28, 2012