Re: [sympy-live] SymPy Live Sphinx extension (#59)

2012-08-08 Thread David Li
I will work on the individual-statement highlighting when I next get a chance. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy-live/pull/59#issuecomment-7582813 -- You received this message because you are subscribed to the Google Groups sympy-patches

Re: [sympy-live] SymPy Live Sphinx extension (#59)

2012-08-08 Thread David Li
Alright, now you can click a line to execute. I am deploying it to the test server. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy-live/pull/59#issuecomment-7602291 -- You received this message because you are subscribed to the Google Groups

Re: [sympy-live] SymPy Live Sphinx extension (#59)

2012-08-08 Thread Aaron Meurer
OK, the execution here seems to work OK. There are just a couple of user interface issues. First, the Run in SymPy Live should now obviously be changed. For one thing, it overlaps with the first prompt. Also, it should probably also be called Run block in SymPy Live. Is it possible to put

Re: [sympy-live] SymPy Live Sphinx extension (#59)

2012-08-08 Thread Aaron Meurer
The copy option for the run whole block just copies the first text, but then executes everything. I think it should just copy everything (and not do any queueing in that case). --- Reply to this email directly or view it on GitHub:

Re: [sympy-live] SymPy Live Sphinx extension (#59)

2012-08-08 Thread Aaron Meurer
The problematic doctest from If you define a circular relationship, you will get a RuntimeError. in the gotchas seems to have disappeared. Is that because of the extension or unrelated? --- Reply to this email directly or view it on GitHub:

Re: [sympy-live] SymPy Live Sphinx extension (#59)

2012-08-08 Thread David Li
Yes, I think that would be due to the extension. I will refactor the code a bit next chance I get and make those changes/fix those bugs. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy-live/pull/59#issuecomment-7603357 -- You received this message because

Re: [sympy-live] SymPy Live Sphinx extension (#59)

2012-08-08 Thread Aaron Meurer
See https://github.com/sympy/sympy/pull/1474 for the collapsible sidebar. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy-live/pull/59#issuecomment-7604421 -- You received this message because you are subscribed to the Google Groups sympy-patches group. To

Re: [sympy-live] SymPy Live Sphinx extension (#59)

2012-08-07 Thread David Li
Okay, I've reverted to the old backend and changed the Sphinx frontend. I am pushing the changes... Also, there seems to be a bug in the normal backend: the code c, d = var('c d') print c c print d d def ctimesd(): ... ... This function returns

Re: [sympy-live] SymPy Live Sphinx extension (#59)

2012-08-07 Thread Aaron Meurer
It appears that defining a function deletes all user-defined names. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy-live/pull/59#issuecomment-7571543 -- You received this message because you are subscribed to the Google Groups sympy-patches group. To post

Re: [sympy-live] SymPy Live Sphinx extension (#59)

2012-08-07 Thread Aaron Meurer
I think the problem is in the split function. It should not split out the last line in a function definition. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy-live/pull/59#issuecomment-7571871 -- You received this message because you are subscribed to the

Re: [sympy-live] SymPy Live Sphinx extension (#59)

2012-08-07 Thread Aaron Meurer
So I *think* the correct solution is to check in split if the last line begins with whitespace, and to not include it in the eval code if it does. But I'm still trying to understand the code here, so it may not be right. --- Reply to this email directly or view it on GitHub:

Re: [sympy-live] SymPy Live Sphinx extension (#59)

2012-08-07 Thread Aaron Meurer
Well, that doesn't seem to work. So I'm at a loss here. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy-live/pull/59#issuecomment-7572186 -- You received this message because you are subscribed to the Google Groups sympy-patches group. To post to this

Re: [sympy-live] SymPy Live Sphinx extension (#59)

2012-08-07 Thread Aaron Meurer
Anyway, you don't have to fix that here. It's a bug in master, not a regression, so it doesn't have any barring on this PR. Are you planning to do more here, or should I review to merge? --- Reply to this email directly or view it on GitHub:

Re: [sympy-live] SymPy Live Sphinx extension (#59)

2012-08-06 Thread David Li
Okay, now it should print each prompt interspersed with the results. I've pushed the changes to the server. Also, I fixed the bug where preexecs weren't getting evaluated...turns out I was overwriting a variable. --- Reply to this email directly or view it on GitHub:

Re: [sympy-live] SymPy Live Sphinx extension (#59)

2012-08-06 Thread Aaron Meurer
You accidentally removed app.yaml --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy-live/pull/59#issuecomment-7542001 -- You received this message because you are subscribed to the Google Groups sympy-patches group. To post to this group, send email to

Re: [sympy-live] SymPy Live Sphinx extension (#59)

2012-08-06 Thread Aaron Meurer
Can you update the live demo? --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy-live/pull/59#issuecomment-7542027 -- You received this message because you are subscribed to the Google Groups sympy-patches group. To post to this group, send email to

Re: [sympy-live] SymPy Live Sphinx extension (#59)

2012-08-06 Thread David Li
Sorry, fixed. Still some issues: - Code like def ctimesd(): ... ... This function returns whatever c is times whatever d is. ... ... return c*d isn't getting executed properly. - Tracebacks don't show the original prompt --- Reply to this email

Re: [sympy-live] SymPy Live Sphinx extension (#59)

2012-08-06 Thread Aaron Meurer
OK, so I guess the behavior should be like this: If there are multiple things in the same input, it should only print the last one. This is how even the IPython notebook works. For the Sphinx extension, it should parse the `` and `...` to determine what should be executed as one block.

[sympy-live] SymPy Live Sphinx extension (#59)

2012-08-04 Thread David Li
- Adds a Show/Hide SymPy Live Shell button to the bottom-right corner of the docs. - Clicking it shows a shell that takes up about 600x500px of space. - The Fullscreen button normally in SymPy Live now redirects to SymPy Live. - Settings are in a dropdown under the shell. - Hovering