#11287 closed enhancement (fixed)
Interface to runsnake and import_statements
Reported by: | nthiery | Owned by: | tbd |
---|---|---|---|
Priority: | major | Milestone: | sage-4.7.2 |
Component: | performance | Keywords: | runsnake, prun, profiling, days30 |
Cc: | sage-combinat, saliola, aschilling, novoselt | Merged in: | sage-4.7.2.alpha0 |
Authors: | Nicolas M. Thiéry | Reviewers: | Franco Saliola, Simon King, Christian Stump |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | #11251 | Stopgaps: |
Description (last modified by )
This patch adds two developper tools in sage.misc.dev_tools:
- An interface to the graphical profiler
runsnake
. See e.g. the attached image which shows a graphical profile of listing all the elements of SymmetricGroup?(3), and highlights the current GAP interface issue with select on certain linux kernels. See also #11291.
- A function import_statements:
sage: import_statements(WeylGroup, lazy_attribute) from sage.combinat.root_system.weyl_group import WeylGroup from sage.misc.lazy_attribute import lazy_attribute sage: import_statements(WeylGroup, lazy_attribute, lazy=True) from sage.misc.lazy_import import lazy_import lazy_import('sage.combinat.root_system.weyl_group', 'WeylGroup') lazy_import('sage.misc.lazy_attribute', 'lazy_attribute')
To achieve this, it also extracts a function get_main_globals out of inject_variables.
Attachments (2)
Change History (46)
Changed 6 years ago by
comment:1 Changed 6 years ago by
- Cc saliola added
- Description modified (diff)
- Status changed from new to needs_review
comment:2 Changed 6 years ago by
- Description modified (diff)
comment:3 Changed 6 years ago by
- Description modified (diff)
comment:4 Changed 6 years ago by
As I use very very heavily runsnake this last mouth, I try your patch as soon as I received the corresponding mail.
Applying the patch (toward the combinat queue), runsnake works well on my machine in sage. (I had to remove my old runsnake function in .sage/init.sage and re-install RunSankeRun? vis easy-install).
Ubuntu 04-11 32bits on a MacBook4,1. Not a review, just a test and some info...
+1 to get the feature in Sage!
comment:5 follow-up: ↓ 6 Changed 6 years ago by
- Reviewers changed from Franco Saliola to Franco Saliola, Simon King
Runsnake
Can you give at least a slight hint how to use runsnake? I used easy_install inside a Sage shell, but apparently something (wxPython?) is missing. I thought easy_install is supposed to take care of dependencies.
Also, I think you should give examples how to use the profiler from within Sage. From runsnake's documentation, I guess one does something like
sage: import cProfile sage: cProfile.run'L=[singular(i)==gap(i)==i+1 for i in range(2000)]' cProfile.run cProfile.runctx sage: cProfile.runctx('L=[singular(i)==gap(i)==i+1 for i in range(2000)]', globals(),locals(), filename="OpenGLContext.profile" )
but what afterwards?
Does one need to leave Sage, start a Sage shell, and then open OpenGLContext.profile
by runsnake OpenGLContext.profile
? (That step did not work for me, it seems I need to get wx)
Can one use it without leaving Sage?
import_statements
That tool seems useful. I doubt, however, that it is robust enough. I see that you ask for the __module__
attribute. It is not always present, at least for methods or for extension classes that forgot to initialise the category:
sage: from sage.structure.element import Element sage: Element.parent.__module__ --------------------------------------------------------------------------- AttributeError Traceback (most recent call last) /mnt/local/king/SAGE/sage-4.7.alpha5/<ipython console> in <module>() AttributeError: 'method_descriptor' object has no attribute '__module__' sage: P.<x,y> = QQ[] sage: P.__module__ --------------------------------------------------------------------------- AttributeError Traceback (most recent call last) /mnt/local/king/SAGE/broken/devel/sage-main/<ipython console> in <module>() /mnt/local/king/SAGE/broken/local/lib/python2.6/site-packages/sage/structure/parent.so in sage.structure.parent.Parent.__getattr__ (sage/structure/parent.c:5618)() /mnt/local/king/SAGE/broken/local/lib/python2.6/site-packages/sage/structure/parent.so in sage.structure.parent.raise_attribute_error (sage/structure/parent.c:2636)() AttributeError: 'sage.rings.polynomial.multi_polynomial_libsingular.MPolynomialRing_libsingular' object has no attribute '__module__'
Worse, the __module__
attribute can be misleading. Unfortunately, it is particularly misleading if you initialise the category of a parent:
sage: P._is_category_initialized() False sage: P._init_category_(Algebras(QQ)) sage: P.__module__ 'sage.categories.algebras' sage: import_statements(P) from sage.categories.category import CommutativeAlgebras.parent_class
That import statement yields a syntax error.
Also, I recall that sometimes the __module__
attribute is None
(but I can't remember a typical example).
I would recommend to use the tools from sage.misc.sageinspect, but only after applying #9976, which considerably extends the capabilities of finding source files (here: sage/rings/polynomial/multi_polynomial_libsingular.pyx), from which one can deduce the import statement.
I am not sure if this is enough to make it "needs work", though.
Cheers, Simon
comment:6 in reply to: ↑ 5 ; follow-up: ↓ 8 Changed 6 years ago by
Hello! I just posted a reviewer's patch that addresses some documentation issues. The docstrings for the Profiler module needed to be fixed so that they rendered correctly in the reference manual.
Note that this patch depends on #11251, since you are using the new todo directive that is implemented in that ticket.
I am willing to give this a positive review, provided my changes are acceptable.
Let me try to address Simon's comments:
Replying to SimonKing:
Runsnake
Can you give at least a slight hint how to use runsnake?
There is an example on how to use it provided in the documentation:
EXAMPLES:: sage: runsnake("list(SymmetricGroup(3))") # optional - requires runsnake
The expected behaviour is that a window pops up (see the screenshot).
Does one need to leave Sage, start a Sage shell, and then open
OpenGLContext.profile
byrunsnake OpenGLContext.profile
? (That step did not work for me, it seems I need to get wx)Can one use it without leaving Sage?
Yes. One needs only run the function in the example above. This function initializes the profiler and (doing essentially what you wrote above), and then launches runsnake.
import_statements
That tool seems useful.
Agreed. It looks very nice.
Worse, the
__module__
attribute can be misleading. Unfortunately, it is particularly misleading if you initialise the category of a parent:
This issue is documented in a "todo" block, but this todo block does not
appear in the command-line view of the documentation. So that's probably why
you didn't see it (it does appear if you ask for source
import_statements??
). This is not a problem with this ticket though, but
with the todo directive; see #11251.
I would recommend to use the tools from sage.misc.sageinspect, but only after applying #9976, which considerably extends the capabilities of finding source files (here: sage/rings/polynomial/multi_polynomial_libsingular.pyx), from which one can deduce the import statement.
Perhaps this might be a good idea and can address the issue that the
__module__
attribute is not always present. I'll leave it to Nicolas to
comment on this.
comment:7 Changed 6 years ago by
- Keywords days30 added
comment:8 in reply to: ↑ 6 Changed 6 years ago by
Replying to saliola:
Hello! I just posted a reviewer's patch that addresses some documentation issues. The docstrings for the Profiler module needed to be fixed so that they rendered correctly in the reference manual. ... I am willing to give this a positive review, provided my changes are acceptable.
Positive review on your review patch, assuming the buildbot goes green.
I would recommend to use the tools from sage.misc.sageinspect, but only after applying #9976, which considerably extends the capabilities of finding source files (here: sage/rings/polynomial/multi_polynomial_libsingular.pyx), from which one can deduce the import statement.
Perhaps this might be a good idea and can address the issue that the
__module__
attribute is not always present. I'll leave it to Nicolas to comment on this.
If you know how to improve this on top of your head, please go ahead! Otherwise, I'll vote for just pushing this as is, and fix it later. It is annoying if this method is not robust, but not dangerous, and we want to let it be used to get feedback.
Cheers,
Nicolas
comment:9 Changed 6 years ago by
I thought that the improvements to sageinspect from #9976 would make things robust enough, so that sage_getfile could be used to extract the information on where to import stuff from.
But with or without #9976, we have
sage: P.<x,y> = QQ[] sage: P?? Error getting source: could not find class definition ... Docstring [source file open failed]: File: sage/rings/polynomial/multi_polynomial_libsingular.pyx (starting at line 229) Construct a multivariate polynomial ring subject to the following conditions: INPUT: * ``base_ring`` - base ring (must be either GF(q), ZZ, ZZ/nZZ, QQ or absolute number field) * ``n`` - number of variables (must be at least 1) * ``names`` - names of ring variables, may be string of list/tuple * ``order`` - term order (default: ``degrevlex``)
The "Error getting source: could not find class definition" is annoying. Note that the doc string above gives the correct location and line number! So, sage.misc._extract_embedded_position should be able to do the job, but it isn't.
I think improving the introspection further should be on a different ticket, on top of #9976.
Cheers, Simon
comment:10 Changed 6 years ago by
I just opened #11298 (ready for review).
With it, we obtain:
sage: from sage.misc.sageinspect import sage_getsourcelines, sage_getfile sage: P.<x,y> = QQ[] sage: P._init_category_(Algebras(QQ)) sage: P.__module__ 'sage.categories.algebras' sage: sage_getfile(P) '/mnt/local/king/SAGE/sage-4.7.alpha5/devel/sage/sage/rings/polynomial/multi_polynomial_libsingular.pyx' sage: lines, lineno = sage_getsourcelines(P) sage: lines[0] 'cdef class MPolynomialRing_libsingular(MPolynomialRing_generic):\n'
It should be easy to extract the module name from the file name (here: sage.rings.polynomial.multi_polynomial_libsingular
, not sage.categories.algebras
). Moreover, it should be easy to extract the correct class name from the first line of the source code (ClassName
, not ClassName_with_category
or SomeCategory.parent_class
, which would not work in an import statement).
Cheers, Simon
comment:11 follow-up: ↓ 12 Changed 6 years ago by
Ubuntu/Debian? RunSnakeRun
install
Along with a few missteps, this is what I needed to do to have RunSnakeRun
on an Ubuntu Lucid system. Presumably this will work for other Debian-based distributions.
wxPython
Follow instructions at:
http://wiki.wxpython.org/InstallingOnUbuntuOrDebian
Python easy_install setup program, pstats profiling module, RunSnakeRun
itself
sudo apt-get install python-setuptools python-profiler easy_install RunSnakeRun
comment:12 in reply to: ↑ 11 ; follow-up: ↓ 13 Changed 6 years ago by
Replying to rbeezer:
Ubuntu/Debian? RunSnakeRun
install, part 2
There are better instructions in the doc string. wxPython is an Ubuntu package (python-wxgtk2.8).
comment:13 in reply to: ↑ 12 Changed 6 years ago by
Replying to rbeezer:
There are better instructions in the doc string. wxPython is an Ubuntu package (python-wxgtk2.8).
OK, but my personal problem is that I am not root for the computer in my office. So, installing an Ubuntu package seems non-trivial to me.
I tried installing wxpython from sources and also tried the experimental wxpython spkg. In both cases, the problem was that gtk+ has not been found, although it is globally installed. The sysadmin tries to help me, but I don't know if it will work eventually.
comment:14 follow-up: ↓ 15 Changed 6 years ago by
Could it be that the Sage shell messes up? Namely, I downloaded wxPython
sources. When I do ./configure
in a Python shell, it complains about not finding gtk. If I do it on a usual shell, it works. I am now trying make und make install outside the python shell. I hope that it suffices to have wxpython in the path, i.e., it is not assumed that it was installed in the Sage scope?
Best regards, Simon
comment:15 in reply to: ↑ 14 Changed 6 years ago by
Replying to SimonKing:
Could it be that the Sage shell messes up? Namely, I downloaded
wxPython
sources. When I do./configure
in a Python shell,
I meant to write "Sage shell".
comment:16 Changed 6 years ago by
Unfortunately, after doing easy_install RunSnakeRun?, things still do not work for me:
sage: runsnake("list(SymmetricGroup(3))") sage: Traceback (most recent call last): File "/mnt/local/king/SAGE/sage-4.7.alpha5/local/bin/runsnake", line 5, in <module> from pkg_resources import load_entry_point File "/usr/lib/python2.6/dist-packages/pkg_resources.py", line 2671, in <module> working_set.require(__requires__) File "/usr/lib/python2.6/dist-packages/pkg_resources.py", line 654, in require needed = self.resolve(parse_requirements(requirements)) File "/usr/lib/python2.6/dist-packages/pkg_resources.py", line 552, in resolve raise DistributionNotFound(req) pkg_resources.DistributionNotFound: RunSnakeRun==2.0.1b6
I did not do sudo apt-get install python-setuptools python-profiler
, as I am not root. Is that the missing bit?
comment:17 follow-up: ↓ 22 Changed 6 years ago by
PS: If I leave sage, but remain in a sage shell, and try runsnake OpenGLContext.profile
then the error says ImportError: No module named wx
. So, apparently the problem is that I really need to install wxpython inside a Sage shell, but I can't since the Sage shell can not find stuff that is known to the rest of the world. End of the story. I have a lecture to prepare.
comment:18 follow-up: ↓ 19 Changed 6 years ago by
Hi, I pushed a review concerning two minor things I ran into:
on the sage command line, spaces do not cause trouble, but the profiler doesn't like them:
sage: runsnake(' 3') ... IndentationError: unexpected indent (<string>, line 1)
Now, spaces are deleted (or does anyone see problems occurring from that?).
The second is that sage accepts "^{", while python doesn't, so these are replaced by "". }
sage: runsnake('QQ(3)^2') ... RuntimeError: Use ** for exponentiation, not '^', which means xor in Python, and has the wrong precedence.
You can fold the changes if you like, or remove them otherwise...
Btw: the patch doesn't apply on a plain 4.6.2.
Best, Christian
comment:19 in reply to: ↑ 18 ; follow-up: ↓ 20 Changed 6 years ago by
Replying to stumpc5:
The second is that sage accepts "^{", while python doesn't, so these are replaced by "". }
What about calling the Sage preparser, instead?
comment:20 in reply to: ↑ 19 Changed 6 years ago by
Thanks Christian for your improvements.
Replying to SimonKing:
Replying to stumpc5:
The second is that sage accepts "^{", while python doesn't, so these are replaced by "". }
What about calling the Sage preparser, instead?
+1.
Christian: I won't be able to work on this before a couple day. Could you give it a shot?
Also, I would tend to only strip spaces before and after the command; otherwise, one might inadvertently screw up spaces in an embedded string.
Cheers,
comment:21 follow-ups: ↓ 23 ↓ 25 Changed 6 years ago by
- Cc aschilling added
- Status changed from needs_review to needs_work
MacOS users (Anne!): The runsnake run author was kind enough to add a workaround in runsnake to make it work out of the box on MacOS. He would like to get it tested. Could you try out the attached devel version of runsnake, and see if:
tar zxvf runsnakerun-2.0.1-devel.tgz cd runsnakerun-2.0.1-devel sudo python setup.py install runsnake
does the job for you?
Anne: you may want to first remove the current runsnake scripts from your machine; if I recall correctly, they are in /usr/local/bin/runsnake and runsnake32bit.
comment:22 in reply to: ↑ 17 Changed 6 years ago by
Replying to SimonKing:
PS: If I leave sage, but remain in a sage shell, and try
runsnake OpenGLContext.profile
then the error saysImportError: No module named wx
. So, apparently the problem is that I really need to install wxpython inside a Sage shell, but I can't since the Sage shell can not find stuff that is known to the rest of the world. End of the story. I have a lecture to prepare.
The problem is that the Sage shell sets PYTHONPATH and PYTHONLIB. You may try to work around this by using
python -E runsnake ...
which tells Python to ignore those variables. The runsnake command inside Sage does just that, for precisely that reason.
comment:23 in reply to: ↑ 21 ; follow-up: ↓ 24 Changed 6 years ago by
Replying to nthiery:
MacOS users (Anne!): The runsnake run author was kind enough to add a workaround in runsnake to make it work out of the box on MacOS. He would like to get it tested. Could you try out the attached devel version of runsnake, and see if:
tar zxvf runsnakerun-2.0.1-devel.tgz cd runsnakerun-2.0.1-devel sudo python setup.py install runsnakedoes the job for you?
Anne: you may want to first remove the current runsnake scripts from your machine; if I recall correctly, they are in /usr/local/bin/runsnake and runsnake32bit.
Where is runsnakerun-2.0.1-devel.tgz attached?
Cheers,
Anne
comment:24 in reply to: ↑ 23 Changed 6 years ago by
Where is runsnakerun-2.0.1-devel.tgz attached?
Ah shoot, I had not noticed that my attachement was too big and had been refused by trac. Ok, there it is:
http://sage.math.washington.edu/home/nthiery/runsnakerun-2.0.1-devel.tgz
Cheers,
Nicolas
comment:25 in reply to: ↑ 21 ; follow-up: ↓ 26 Changed 6 years ago by
Replying to nthiery:
MacOS users (Anne!): The runsnake run author was kind enough to add a workaround in runsnake to make it work out of the box on MacOS. He would like to get it tested. Could you try out the attached devel version of runsnake, and see if:
I gave it a try, and have some comments.
- The patch does not apply cleanly on top of sage-4.7.rc1; in the file
doc/en/reference/misc.rst
there is a conflict because ofclassgraph
. So your patch needs a very trivial rebase.
- I checked and it seems that wxPython is already installed on two different Mac OS X systems I tested. Is this a standard python library on Mac OSX? So there might be no need to install it.
- Just running runsnake does not work:
Traceback (most recent call last): File "/usr/local/bin/runsnake", line 8, in <module> load_entry_point('RunSnakeRun==2.0.1', 'gui_scripts', 'runsnake')() File "/Library/Python/2.6/site-packages/RunSnakeRun-2.0.1-py2.6.egg/runsnakerun/macshim.py", line 7, in macshim env={"VERSIONER_PYTHON_PREFER_32_BIT":"yes"} File "/System/Library/Frameworks/Python.framework/Versions/2.6/lib/python2.6/subprocess.py", line 444, in call return Popen(*popenargs, **kwargs).wait() File "/System/Library/Frameworks/Python.framework/Versions/2.6/lib/python2.6/subprocess.py", line 595, in __init__ errread, errwrite) File "/System/Library/Frameworks/Python.framework/Versions/2.6/lib/python2.6/subprocess.py", line 1106, in _execute_child raise child_exception OSError: [Errno 2] No such file or directory
So I started poking around in macshim.py
. It seems the problem is with the env dictionary
env={"VERSIONER_PYTHON_PREFER_32_BIT":"yes"}
The documentation for the subprocess module says that env must provide any variables required for the program to execute. This includes the path; so one either needs to provide the path to runsnake32 as an environment variable, or just give the complete path to runsnake32. Here is a modified version of macshim.py
that works for me:
def macshim(): """Shim to run 32-bit on 64-bit mac as a sub-process""" import subprocess, sys subprocess.call([ sys.argv[0]+'32' ]+sys.argv[1:], env={"VERSIONER_PYTHON_PREFER_32_BIT":"yes"} )
comment:26 in reply to: ↑ 25 ; follow-up: ↓ 28 Changed 6 years ago by
Replying to saliola:
Replying to nthiery: I gave it a try, and have some comments.
- The patch does not apply cleanly on top of sage-4.7.rc1; in the file
doc/en/reference/misc.rst
there is a conflict because ofclassgraph
. So your patch needs a very trivial rebase.
It depends on #11108 which is already merged in 4.7.1. I added this in the dependency list.
- I checked and it seems that wxPython is already installed on two different Mac OS X systems I tested. Is this a standard python library on Mac OSX? So there might be no need to install it.
Good to know.
- Just running runsnake does not work:
Thanks. I'll transmit this info to the runsnake developer!
comment:27 Changed 6 years ago by
- Dependencies set to #11108
comment:28 in reply to: ↑ 26 Changed 6 years ago by
Hi Franco,
- Just running runsnake does not work:
Thanks. I'll transmit this info to the runsnake developer!
Mike just updated runsnake using your suggestion. I updated accordingly:
http://sage.math.washington.edu/home/nthiery/runsnakerun-2.0.1-devel.tgz
Franco: please check!
Anne: you may want to give it another try. I don't understand why the runsnake command did not get installed in your previous attempt though.
comment:29 follow-up: ↓ 31 Changed 6 years ago by
I get the same problem as yesterday:
d238:runsnakerun-2.0.1-devel anne$ sudo python setup.py install /Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown distribution option: 'entry_points' warnings.warn(msg) /Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown distribution option: 'install_requires' warnings.warn(msg) /Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown distribution option: 'zip_safe' warnings.warn(msg) running install running build running build_py creating build/lib creating build/lib/runsnakerun copying runsnakerun/__init__.py -> build/lib/runsnakerun copying runsnakerun/_meliaejson.py -> build/lib/runsnakerun copying runsnakerun/dumbprofile.py -> build/lib/runsnakerun copying runsnakerun/hotshotreader.py -> build/lib/runsnakerun copying runsnakerun/listviews.py -> build/lib/runsnakerun copying runsnakerun/macshim.py -> build/lib/runsnakerun copying runsnakerun/meliaeadapter.py -> build/lib/runsnakerun copying runsnakerun/meliaeloader.py -> build/lib/runsnakerun copying runsnakerun/pstatsadapter.py -> build/lib/runsnakerun copying runsnakerun/pstatsloader.py -> build/lib/runsnakerun copying runsnakerun/runsnake.py -> build/lib/runsnakerun running install_lib copying build/lib/runsnakerun/macshim.py -> /Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/runsnakerun copying build/lib/runsnakerun/runsnake.py -> /Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/runsnakerun byte-compiling /Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/runsnakerun/macshim.py to macshim.pyc byte-compiling /Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/runsnakerun/runsnake.py to runsnake.pyc running install_egg_info Removing /Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/RunSnakeRun-2.0.1-py2.7.egg-info Writing /Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/RunSnakeRun-2.0.1-py2.7.egg-info d238:runsnakerun-2.0.1-devel anne$ runsnake -bash: runsnake: command not found
Cheers,
Anne
comment:30 Changed 6 years ago by
I just updated my small review patch using now the sage preparser. Here is an example:
sage: runsnake('for x in range(4): print x^2') 0 1 4 9
comment:31 in reply to: ↑ 29 Changed 6 years ago by
Replying to aschilling:
I get the same problem as yesterday:
d238:runsnakerun-2.0.1-devel anne$ sudo python setup.py install ... copying build/lib/runsnakerun/macshim.py -> /Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/runsnakerun copying build/lib/runsnakerun/runsnake.py -> /Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/runsnakerun ... d238:runsnakerun-2.0.1-devel anne$ runsnake -bash: runsnake: command not found
Ah, it seems like the script gets installed in: /Library/Frameworks/Python?.framework/Versions/2.7/lib/python2.7/site-packages/runsnakerun
Could you try to run it?
My best guess is that for some reason python2.7 is not considered as the default Python install on your machine, and thus python scripts are not installed in the general /usr/local/bin.
Cheers,
Nicolas
comment:32 Changed 6 years ago by
- Reviewers changed from Franco Saliola, Simon King to Franco Saliola, Simon King, Christian Stump
- Status changed from needs_work to needs_review
Thanks Franco and Christian for your reviewer patches. I folded them in, improved just a tiny bit the documentation (including Christian's example requiring preparsing), and uploaded it there. It should be good to go.
comment:33 Changed 6 years ago by
- The latest snapshot of the runsnake works as advertised on my MacOSX system.
- Someone introduced a tab into
dev_tools.py
, so the doctests failed. I'm uploading a patch removing the tab. Then all doctests pass. Nicolas: look it over and set this to positive review.
- Simon: Python supports installing modules into a user's home directory. Have you tried that? Here are some instructions: http://docs.python.org/install/index.html#alternate-installation-the-home-scheme
comment:34 follow-up: ↓ 35 Changed 6 years ago by
Note that my patch also removes a line of documentation: the globals
variable is no longer used by the runsnake
command.
comment:35 in reply to: ↑ 34 Changed 6 years ago by
- Status changed from needs_review to positive_review
Replying to saliola:
Note that my patch also removes a line of documentation: the
globals
variable is no longer used by therunsnake
command.
Thanks for the fix. I folded your patch, and am about to set a positive review.
Thanks everybody!
comment:36 follow-up: ↓ 37 Changed 6 years ago by
- Status changed from positive_review to needs_work
Please change the commit message of the patch.
Changed 6 years ago by
comment:37 in reply to: ↑ 36 Changed 6 years ago by
- Dependencies #11108 deleted
- Status changed from needs_work to positive_review
comment:38 follow-up: ↓ 39 Changed 6 years ago by
- Status changed from positive_review to needs_work
There is a problem with the documentation:
dochtml.log:/mnt/usb1/scratch/jdemeyer/merger/sage-4.7.1.alpha3/local/lib/python2.6/site-packages/sage/misc/dev_tools.py:docstring of sage.misc.dev_tools.import_statements:14: (ERROR/3) Unknown directive type "todo". dochtml.log:/mnt/usb1/scratch/jdemeyer/merger/sage-4.7.1.alpha3/local/lib/python2.6/site-packages/sage/misc/profiler.py:docstring of sage.misc.profiler.Profiler:27: (ERROR/3) Unknown directive type "todo".
comment:39 in reply to: ↑ 38 Changed 6 years ago by
- Dependencies set to #11251
Replying to jdemeyer:
There is a problem with the documentation:
dochtml.log:/mnt/usb1/scratch/jdemeyer/merger/sage-4.7.1.alpha3/local/lib/python2.6/site-packages/sage/misc/dev_tools.py:docstring of sage.misc.dev_tools.import_statements:14: (ERROR/3) Unknown directive type "todo". dochtml.log:/mnt/usb1/scratch/jdemeyer/merger/sage-4.7.1.alpha3/local/lib/python2.6/site-packages/sage/misc/profiler.py:docstring of sage.misc.profiler.Profiler:27: (ERROR/3) Unknown directive type "todo".
Ah shoot. This directive is implemented by #11251 which needs review. Ok, I added this in the dependency list, and am going to ping #11251.
Thanks for the report ...
comment:40 Changed 6 years ago by
- Status changed from needs_work to positive_review
comment:41 Changed 6 years ago by
- Milestone changed from sage-4.7.1 to sage-wait
comment:42 Changed 6 years ago by
- Cc novoselt added
comment:43 Changed 6 years ago by
- Merged in set to sage-4.7.2.alpha0
- Resolution set to fixed
- Status changed from positive_review to closed
comment:44 Changed 6 years ago by
- Milestone changed from sage-pending to sage-4.7.2
Screen shot of runsnake in action