Re: [sympy] [Work in Progress] Basic framework for restructured plot. (#1468)

2012-08-07 Thread The Travis Bot
This pull request [fails](http://travis-ci.org/sympy/sympy/builds/2054782) (merged 85cd5a99 into 65b65820). --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1468#issuecomment-7544733 -- You received this message because you are subscribed to the Google

Re: [sympy] combsimp fix (#1462)

2012-08-07 Thread Christopher Smith
Tom, how do you feel about putting in the new multiplication_theorem algorithm that doesn't depend on the order of terms? And how about not doing simplification of the differences? If you don't simplify terms then I *think* the gamma args will sort so that the rationals, for a given residual (arg

Re: [sympy] [Work in Progress] Basic framework for restructured plot. (#1468)

2012-08-07 Thread Aaron Meurer
+The plotting uses an adaptive algorithm which samples recursively to +accurately plot the plot. The adaptive algorithm uses a random point near +the midpoint of two points that has to be further sampled. Hence the same +plots can appear slightly different. + +Usage +

Re: [sympy] [Work in Progress] Basic framework for restructured plot. (#1468)

2012-08-07 Thread Aaron Meurer
+ +The plotting uses an adaptive algorithm which samples recursively to +accurately plot the plot. The adaptive algorithm uses a random point near +the midpoint of two points that has to be further sampled. Hence the same +plots can appear slightly different. + +Usage +

Re: [sympy] [Work in Progress] Basic framework for restructured plot. (#1468)

2012-08-07 Thread Aaron Meurer
+``xlim`` : tuple of two floats, denoting the x - axis limits. + +``ylim`` : tuple of two floats, denoting the y - axis limits. + +Examples + + plot_line(x**2, (x, -5, 5)) + +Multiple plots + plot_line((x**2, (x, -6, 6)), (x, (x, -5, 5))) + +

Re: [sympy] [Work in Progress] Basic framework for restructured plot. (#1468)

2012-08-07 Thread Aaron Meurer
Why create a new file for this? --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1468#issuecomment-7545659 -- 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] combsimp fix (#1462)

2012-08-07 Thread The Travis Bot
This pull request [passes](http://travis-ci.org/sympy/sympy/builds/2055088) (merged 1bce5308 into 65b65820). --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1462#issuecomment-7545954 -- You received this message because you are subscribed to the

Re: Issue 3349 in sympy: gremlin in master

2012-08-07 Thread sympy
Updates: Status: Fixed Comment #7 on issue 3349 by smi...@gmail.com: gremlin in master http://code.google.com/p/sympy/issues/detail?id=3349 (No comment was entered for this change.) -- You received this message because you are subscribed to the Google Groups sympy-patches group. To

[sympy] Fixed an error in kinematic differential equations. (#1469)

2012-08-07 Thread Angadh Nanjangud
@smichr @gilbertgede @moorepants @hazelnusse You can merge this Pull Request by running: git pull https://github.com/angadhn/sympy kanechanges Or you can view, comment on it, or merge it online at: https://github.com/sympy/sympy/pull/1469 -- Commit Summary -- * Fixed an error in

Re: Issue 3349 in sympy: gremlin in master

2012-08-07 Thread sympy
Updates: Status: Valid Comment #8 on issue 3349 by smi...@gmail.com: gremlin in master http://code.google.com/p/sympy/issues/detail?id=3349 The following is being printed in Windows durint testing of ipython with 2.7: sympy\interactive\tests\test_ipython.py[1] WARNING: Readline

Re: [sympy] LagrangesMethod class for sympy.physics.mechanics. (#1460)

2012-08-07 Thread The Travis Bot
This pull request [fails](http://travis-ci.org/sympy/sympy/builds/2056368) (merged 84a5ea6f into 65b65820). --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1460#issuecomment-7550552 -- You received this message because you are subscribed to the Google

Re: [sympy] Fixed an error in kinematic differential equations. (#1469)

2012-08-07 Thread The Travis Bot
This pull request [passes](http://travis-ci.org/sympy/sympy/builds/2056427) (merged 67270875 into 98cc80f0). --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1469#issuecomment-7551074 -- You received this message because you are subscribed to the

Re: [sympy] Fixed an error in kinematic differential equations. (#1469)

2012-08-07 Thread Stefan Krastanov
**SymPy Bot Summary:** :eight_spoked_asterisk: All tests have passed. *Test command:* setup.py test *master hash*: 98cc80f01b82534e243cfb791d5977f18bb2414e *branch hash*: 67270875bd85734785bb24816d2ec3d968e8a960 **Interpreter 1:** :eight_spoked_asterisk: All tests have passed. *Interpreter:*

Re: [sympy] combsimp fix (#1462)

2012-08-07 Thread The Travis Bot
This pull request [passes](http://travis-ci.org/sympy/sympy/builds/2056716) (merged cff42d03 into 65b65820). --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1462#issuecomment-7552075 -- You received this message because you are subscribed to the

Re: [sympy] ImmutableMatrix defaults to MatrixBase operators (#1467)

2012-08-07 Thread Matthew Rocklin
Should I add tests like `isinstance(X + X, ImmutableMatrix)`? This seems odd to me in some way. It doesn't test the mathematics but instead tests nitty-gritty implementation details. --- Reply to this email directly or view it on GitHub:

Re: [sympy] Diagram Layout (#1429)

2012-08-07 Thread Sergiu Ivanov
+ from sympy.categories import Object, NamedMorphism + from sympy.categories import Diagram, DiagramGrid + from sympy import FiniteSet + A = Object(A) + B = Object(B) + C = Object(C) + f = NamedMorphism(A, B, f) + g = NamedMorphism(B, C, g) + diagram =

Re: [sympy] Diagram Layout (#1429)

2012-08-07 Thread The Travis Bot
This pull request [passes](http://travis-ci.org/sympy/sympy/builds/2057075) (merged 47b76408 into 41e968b8). --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1429#issuecomment-7553680 -- You received this message because you are subscribed to the

Re: [sympy] [Work in Progress] Basic framework for restructured plot. (#1468)

2012-08-07 Thread Stefan Krastanov
**SymPy Bot Summary:** :red_circle: There were test failures. @catchmrbharath: Please fix the test failures. *Test command:* setup.py test *master hash*: 98cc80f01b82534e243cfb791d5977f18bb2414e *branch hash*: 85cd5a99c5f32d08e756b0c827c3c5c46086ea65 **Interpreter 1:** :red_circle: There were

Re: [sympy] Diagram Layout (#1429)

2012-08-07 Thread Sergiu Ivanov
@ness01 , thanks for the extra round of review! I have fixed everything you have pointed out, including fixing all whitespace in writing `dicts`. I get the same errors with Python 3.3, but I get them all over SymPy anyway, so I believe they are hardly introduced by my code. --- Reply to this

Re: [sympy] ImmutableMatrix defaults to MatrixBase operators (#1467)

2012-08-07 Thread Stefan Krastanov
**SymPy Bot Summary:** :eight_spoked_asterisk: All tests have passed. *Test command:* setup.py test *master hash*: 98cc80f01b82534e243cfb791d5977f18bb2414e *branch hash*: e3a033ee001d38b4c7e84a3e03ca39d25bfe3f0e **Interpreter 1:** :eight_spoked_asterisk: All tests have passed. *Interpreter:*

Re: [sympy] Diagram Drawing (#1430)

2012-08-07 Thread The Travis Bot
This pull request [passes](http://travis-ci.org/sympy/sympy/builds/2057716) (merged f0e1229b into 98cc80f0). --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1430#issuecomment-7556081 -- You received this message because you are subscribed to the

Re: [sympy] combsimp fix (#1462)

2012-08-07 Thread Tom Bachmann
Chris, if you say this is better and all tests still pass I won't object. Looking at my code again I see how terrible it is :/. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1462#issuecomment-7558167 -- You received this message because you are

Re: [sympy] combsimp fix (#1462)

2012-08-07 Thread Stefan Krastanov
**SymPy Bot Summary:** :eight_spoked_asterisk: All tests have passed. *Test command:* setup.py test *master hash*: 98cc80f01b82534e243cfb791d5977f18bb2414e *branch hash*: cff42d0365a58489349b93a279178dd82ef617f1 **Interpreter 1:** :eight_spoked_asterisk: All tests have passed. *Interpreter:*

Re: [sympy] Diagram Layout (#1429)

2012-08-07 Thread Tom Bachmann
@scolobb solid work! I'll review diagram-drawing as soon as I can. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1429#issuecomment-7558407 -- You received this message because you are subscribed to the Google Groups sympy-patches group. To post to

Re: [sympy] Diagram Drawing (#1430)

2012-08-07 Thread Tom Bachmann
Can you rebase this onto master? It seems the commits from diagram-layout are still shown here... --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1430#issuecomment-7560834 -- You received this message because you are subscribed to the Google Groups

Re: [sympy] [Work in Progress] Basic framework for restructured plot. (#1468)

2012-08-07 Thread The Travis Bot
This pull request [fails](http://travis-ci.org/sympy/sympy/builds/2059543) (merged d9efa1ea into 65b65820). --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1468#issuecomment-7562251 -- You received this message because you are subscribed to the Google

Re: [sympy] Fidelity of quantum states (#1459)

2012-08-07 Thread Stefan Krastanov
**SymPy Bot Summary:** :eight_spoked_asterisk: All tests have passed. *Test command:* setup.py test *master hash*: 3c2c3c755445737074de437f004143792f5057e3 *branch hash*: ac679d1dba3a92e2d1e03a311d2110b696cbce5d **Interpreter 1:** :eight_spoked_asterisk: All tests have passed. *Interpreter:*

Re: [sympy] Diagram Drawing (#1430)

2012-08-07 Thread Sergiu Ivanov
@ness01 , done; sorry, I totally forgot to rebase this branch :-( --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1430#issuecomment-7562441 -- You received this message because you are subscribed to the Google Groups sympy-patches group. To post to

Re: [sympy] Fixed an error in kinematic differential equations. (#1469)

2012-08-07 Thread Angadh Nanjangud
A generalized speed was incorrectly defined. I redid the kinematics and verified this. It doesn't affect the test in anyway. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1469#issuecomment-7562585 -- You received this message because you are

Re: [sympy] Diagram Drawing (#1430)

2012-08-07 Thread Tom Bachmann
On 07.08.2012 19:32, Sergiu Ivanov wrote: @ness01 https://github.com/ness01 , done; sorry, I totally forgot to rebase this branch :-( No problem. I'll start the reviewing tomorrow. --- Reply to this email directly or view it on GitHub:

Re: [sympy] Diagram Drawing (#1430)

2012-08-07 Thread Sergiu Ivanov
No problem. I'll start the reviewing tomorrow. Cool! --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1430#issuecomment-7562688 -- You received this message because you are subscribed to the Google Groups sympy-patches group. To post to this group,

Re: [sympy] ImmutableMatrix defaults to MatrixBase operators (#1467)

2012-08-07 Thread Aaron Meurer
Or you could test for specific matrices. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1467#issuecomment-7562891 -- 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] Diagram Drawing (#1430)

2012-08-07 Thread The Travis Bot
This pull request [passes](http://travis-ci.org/sympy/sympy/builds/2059822) (merged d4bdaa5e into 3c2c3c75). --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1430#issuecomment-7563321 -- You received this message because you are subscribed to the

Re: [sympy] Diagram Drawing (#1430)

2012-08-07 Thread Ondřej Čertík
@ness01, thanks! --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1430#issuecomment-7563450 -- 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] Lazy orthogonal polynomials (#1457)

2012-08-07 Thread Stefan Krastanov
**SymPy Bot Summary:** :eight_spoked_asterisk: All tests have passed. *Test command:* setup.py test *master hash*: 3c2c3c755445737074de437f004143792f5057e3 *branch hash*: 82a723cc03d466ecd6da09896abccdc76e24be4c **Interpreter 1:** :eight_spoked_asterisk: All tests have passed. *Interpreter:*

Re: [sympy] Incremental Schreier-Sims algorithm, and subgroup searching. (#1454)

2012-08-07 Thread david joyner
This patch looks good to me and it passes all tests. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1454#issuecomment-7567618 -- You received this message because you are subscribed to the Google Groups sympy-patches group. To post to this group,

Re: [sympy] Diagram Drawing (#1430)

2012-08-07 Thread Stefan Krastanov
**SymPy Bot Summary:** :eight_spoked_asterisk: All tests have passed. *Test command:* setup.py test *master hash*: 3c2c3c755445737074de437f004143792f5057e3 *branch hash*: d4bdaa5ef2ef792521bbbcbd5a89ccf3efce802e **Interpreter 1:** :eight_spoked_asterisk: All tests have passed. *Interpreter:*

Re: Issue 3182 in sympy: Option to isympy to automatically use Rational for int/int

2012-08-07 Thread sympy
Updates: Labels: NeedsReview Comment #4 on issue 3182 by asmeu...@gmail.com: Option to isympy to automatically use Rational for int/int http://code.google.com/p/sympy/issues/detail?id=3182 See https://github.com/sympy/sympy/pull/1470. -- You received this message because you are

Re: [sympy] Autorat (#1470)

2012-08-07 Thread Aaron Meurer
OK, some tests added. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1470#issuecomment-7569394 -- 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] Incremental Schreier-Sims algorithm, and subgroup searching. (#1454)

2012-08-07 Thread Aaron Meurer
OK, I'll merge it then. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1454#issuecomment-7569421 -- 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] Autorat (#1470)

2012-08-07 Thread The Travis Bot
This pull request [fails](http://travis-ci.org/sympy/sympy/builds/2061827) (merged a8eb22cc into 3c2c3c75). --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1470#issuecomment-7569610 -- You received this message because you are subscribed to the Google

Re: [sympy] Autorat (#1470)

2012-08-07 Thread The Travis Bot
This pull request [fails](http://travis-ci.org/sympy/sympy/builds/2061912) (merged 42fdfe2e into 3c2c3c75). --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1470#issuecomment-7569686 -- You received this message because you are subscribed to the Google

Re: [sympy] Autorat (#1470)

2012-08-07 Thread Aaron Meurer
Huh, apparently binary literals were added in Python 2.6. I did not know that. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1470#issuecomment-7569709 -- You received this message because you are subscribed to the Google Groups sympy-patches group.

Re: [sympy] Diagram Layout (#1429)

2012-08-07 Thread Stefan Krastanov
**SymPy Bot Summary:** :eight_spoked_asterisk: All tests have passed. *Test command:* setup.py test *master hash*: 3c2c3c755445737074de437f004143792f5057e3 *branch hash*: 47b7640842f9d79acbc9f2e50cd80b5880828ece **Interpreter 1:** :eight_spoked_asterisk: All tests have passed. *Interpreter:*

Re: [sympy] Lazy orthogonal polynomials (#1457)

2012-08-07 Thread Aaron Meurer
**[SymPy Bot][sympy-bot] Summary**: :red_circle: There were test failures (merged raoulb/lazy_orthogonal_polynomials (82a723cc03d466ecd6da09896abccdc76e24be4c) into master (c7a4a4a39b9154670f218265e76fd778f6bf487b)). @raoulb: Please fix the test failures. **Build HTML Docs:** :red_circle:

Re: [sympy] Handle the --no-colors option only in the write() function (#1461)

2012-08-07 Thread Aaron Meurer
**[SymPy Bot][sympy-bot] Summary**: :red_circle: There were test failures (merged asmeurer/test_colors (f8e8af88029d992b6267dc0c6fc29f90e436f480) into master (c7a4a4a39b9154670f218265e76fd778f6bf487b)). @asmeurer: Please fix the test failures. **Build HTML Docs:** :red_circle: There were test

Re: [sympy] Diagram Layout (#1429)

2012-08-07 Thread Aaron Meurer
**[SymPy Bot][sympy-bot] Summary**: :red_circle: There were test failures (merged scolobb/ct1-diagram-layout (47b7640842f9d79acbc9f2e50cd80b5880828ece) into master (c7a4a4a39b9154670f218265e76fd778f6bf487b)). @scolobb: Please fix the test failures. **Build HTML Docs:** :red_circle: There were

Re: [sympy] Diagram Layout (#1429)

2012-08-07 Thread Aaron Meurer
@scolobb I think these doc failures are due to this branch. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1429#issuecomment-7570118 -- 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 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] Autorat (#1470)

2012-08-07 Thread Ondřej Čertík
+from tokenize import generate_tokens, untokenize, NUMBER, NAME, OP +from StringIO import StringIO + +result = [] +g = generate_tokens(StringIO(s).readline) # tokenize the string +for toknum, tokval, _, _, _ in g: +if toknum == NUMBER and '.' not in tokval: #

Re: [sympy] Autorat (#1470)

2012-08-07 Thread Ondřej Čertík
+# Check the cell for syntax errors. This way, the syntax error +# will show the original input, not the transformed input. The +# downside here is that IPython magic like %timeit will not work +# with transformed input (but on the other hand,

Re: [sympy] Autorat (#1470)

2012-08-07 Thread Ondřej Čertík
+Example +=== + + from sympy.interactive.session import int_to_Integer + from sympy import Integer + s = '1.2 + 1/2 - 0x12 + a1' + int_to_Integer(s) +'1.2 +Integer (1 )/Integer (2 )-Integer (0x12 )+a1 ' + s = 'print (1/2)' + int_to_Integer(s) +

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] Autorat (#1470)

2012-08-07 Thread Christopher Smith
@@ -129,6 +129,27 @@ COMMAND LINE OPTIONS example, if you define a function in isympy with an undefined Symbol, it will not work. +See also the -i and -I options. + +-i, --int-to-Integer + +Automatically wrap int literals with Integer. This makes it so that +

Re: [sympy] Diagram Layout (#1429)

2012-08-07 Thread Ondřej Čertík
But otherwise great job on the patch. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1429#issuecomment-7571681 -- 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] Kinetic and potential energy functionality. (#1407)

2012-08-07 Thread Ondřej Čertík
@angadhn, can you please address all Aaron's and Jason's comments above? E.g. things about the isinstance as well as the +=. Then we can merge it. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1407#issuecomment-7571749 -- You received this message

Re: [sympy] LagrangesMethod class for sympy.physics.mechanics. (#1460)

2012-08-07 Thread Ondřej Čertík
+below_eye = zeros(n+m,n) +A = (self.mass_matrix).col_join((self._m_cd).row_join(zeros(m,m))) +below_I = below_eye.row_join(A) +return I.col_join(below_I) +else: + A = row1.col_join(row2) + return A + +

Re: [sympy] LagrangesMethod class for sympy.physics.mechanics. (#1460)

2012-08-07 Thread Ondřej Čertík
+# of the string fixed frame is redefined as 'LagrangesMethod' doesn't require +# generalized speeds, per se. (Lagrangian mechanics requires 'simple' +# generalized speeds) +A.orientnew('A', 'Axis', [q, N.z]) +A.set_ang_vel(N, qd *A.z) +P.v2pt_theory(O,N,A) +T

Re: [sympy] LagrangesMethod class for sympy.physics.mechanics. (#1460)

2012-08-07 Thread Ondřej Čertík
+The system's mass matrix + +forcing : Matrix +The system's forcing vector + +mass_matrix_full : Matrix +The mass matrix for the qdot's, qdoubledot's, and the +lagrange multipliers (lam) + +forcing_full : Matrix +The forcing vector for

Re: [sympy] Kinetic and potential energy functionality. (#1407)

2012-08-07 Thread Angadh Nanjangud
Hi @certik, I believe I have already addressed the issues. https://github.com/angadhn/sympy/commit/53b6bc2d1f2dd99f6d27a4c154481fa463b0a2ec --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1407#issuecomment-7571850 -- You received this message because

Re: [sympy] Fixed an error in kinematic differential equations. (#1469)

2012-08-07 Thread Christopher Smith
I verified that the test is being run and that it doesn't matter. (You can even remove the cos() denominator altogether and it still passes. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1469#issuecomment-7571857 -- You received this message because

Re: [sympy] LagrangesMethod class for sympy.physics.mechanics. (#1460)

2012-08-07 Thread Ondřej Čertík
You seem to have some trailing whitespace problems there: http://travis-ci.org/#!/sympy/sympy/jobs/2056371 --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1460#issuecomment-7571865 -- You received this message because you are subscribed to the Google

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] Kinetic and potential energy functionality. (#1407)

2012-08-07 Thread Ondřej Čertík
First one note for any PR just keep adding patches to your branch, so that we can quickly see what has changed and one can read the whole PR as a history or story. If you rebase, then suddenly all the test results stop having any sense. Only at the very end, when the PR is ready to go in,

Re: [sympy] Autorat (#1470)

2012-08-07 Thread Aaron Meurer
+from tokenize import generate_tokens, untokenize, NUMBER, NAME, OP +from StringIO import StringIO + +result = [] +g = generate_tokens(StringIO(s).readline) # tokenize the string +for toknum, tokval, _, _, _ in g: +if toknum == NUMBER and '.' not in tokval: #

Re: [sympy] Kinetic and potential energy functionality. (#1407)

2012-08-07 Thread Ondřej Čertík
for e in bodylist: if isinstance(e, (RigidBody, Particle)): -lm_individual = e.linearmomentum(frame) -linearmomentum_sys = linearmomentum_sys + lm_individual +lm_individual = e.linear_momentum(frame) +

Re: [sympy] Autorat (#1470)

2012-08-07 Thread Aaron Meurer
+from tokenize import generate_tokens, untokenize, NUMBER, NAME, OP +from StringIO import StringIO + +result = [] +g = generate_tokens(StringIO(s).readline) # tokenize the string +for toknum, tokval, _, _, _ in g: +if toknum == NUMBER and '.' not in tokval: #

Re: [sympy] Kinetic and potential energy functionality. (#1407)

2012-08-07 Thread Ondřej Čertík
10*N.x + 500*N.y +bodylist = list(bodylist) if not isinstance(bodylist, list): @angadhn, this comment still applies I think. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1407/files#r1328300 -- You received this message

Re: [sympy] Kinetic and potential energy functionality. (#1407)

2012-08-07 Thread Ondřej Čertík
+ from sympy.physics.mechanics import Point, Particle, ReferenceFrame + from sympy.physics.mechanics import RigidBody, outer, potential_energy + from sympy import symbols + M, m, g, h = symbols('M m g h') + N = ReferenceFrame('N') + O = Point('O') +

Re: [sympy] Autorat (#1470)

2012-08-07 Thread Ondřej Čertík
+from tokenize import generate_tokens, untokenize, NUMBER, NAME, OP +from StringIO import StringIO + +result = [] +g = generate_tokens(StringIO(s).readline) # tokenize the string +for toknum, tokval, _, _, _ in g: +if toknum == NUMBER and '.' not in tokval: #

Re: [sympy] Autorat (#1470)

2012-08-07 Thread Aaron Meurer
+Example +=== + + from sympy.interactive.session import int_to_Integer + from sympy import Integer + s = '1.2 + 1/2 - 0x12 + a1' + int_to_Integer(s) +'1.2 +Integer (1 )/Integer (2 )-Integer (0x12 )+a1 ' + s = 'print (1/2)' + int_to_Integer(s) +

Re: [sympy] Autorat (#1470)

2012-08-07 Thread Aaron Meurer
@@ -129,6 +129,27 @@ COMMAND LINE OPTIONS example, if you define a function in isympy with an undefined Symbol, it will not work. +See also the -i and -I options. + +-i, --int-to-Integer + +Automatically wrap int literals with Integer. This makes it so that +

Re: [sympy] Kinetic and potential energy functionality. (#1407)

2012-08-07 Thread Ondřej Čertík
Looks like you rebased again... --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1407#issuecomment-7572129 -- 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-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] Kinetic and potential energy functionality. (#1407)

2012-08-07 Thread The Travis Bot
This pull request [passes](http://travis-ci.org/sympy/sympy/builds/2062711) (merged 1ba125ec into bd49e6a0). --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1407#issuecomment-7572588 -- You received this message because you are subscribed to the

Re: [sympy] Autorat (#1470)

2012-08-07 Thread The Travis Bot
This pull request [passes](http://travis-ci.org/sympy/sympy/builds/2062721) (merged c1aaae19 into 3c2c3c75). --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1470#issuecomment-7572864 -- You received this message because you are subscribed to the

Re: [sympy] LagrangesMethod class for sympy.physics.mechanics. (#1460)

2012-08-07 Thread Gilbert Gede
+ +self.lam_vec = Matrix([]) + + +# Creating the qs, qdots and qdoubledots + +q_list = list(q_list) +if not isinstance(q_list, list): +raise TypeError('Generalized coords. must be supplied in a list') +self._q = q_list +

Re: [sympy] Kinetic and potential energy functionality. (#1407)

2012-08-07 Thread Angadh Nanjangud
Must've done that again mistakenly. Apologies. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1407#issuecomment-7573127 -- You received this message because you are subscribed to the Google Groups sympy-patches group. To post to this group, send

Re: [sympy] Kinetic and potential energy functionality. (#1407)

2012-08-07 Thread Jason Moore
+ + from sympy.physics.mechanics import Particle, Point, outer + from sympy.physics.mechanics import RigidBody, ReferenceFrame + from sympy import symbols + b = ReferenceFrame('b') + M, g, h = symbols('M g h') + P = Point('P') + I =

Re: [sympy] Kinetic and potential energy functionality. (#1407)

2012-08-07 Thread The Travis Bot
This pull request [passes](http://travis-ci.org/sympy/sympy/builds/2063189) (merged caa2aa88 into bd49e6a0). --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1407#issuecomment-7573678 -- You received this message because you are subscribed to the

Re: [sympy] Kinetic and potential energy functionality. (#1407)

2012-08-07 Thread Gilbert Gede
if not isinstance(bodylist, list): raise TypeError('System elements must be supplied as a list') else: -linearmomentum_sys = 0 +linear_momentum_sys = 0 Would returning S(0) here be better, to get a SymPy object? I've had other difficulties in mechanics

Re: [sympy] Kinetic and potential energy functionality. (#1407)

2012-08-07 Thread Gilbert Gede
if not isinstance(bodylist, list): raise TypeError('System elements must be supplied as a list') else: -angularmomentum_sys = 0 +angular_momentum_sys = 0 Same comment about S(0) --- Reply to this email directly or view it on GitHub:

Re: [sympy] LagrangesMethod class for sympy.physics.mechanics. (#1460)

2012-08-07 Thread Gilbert Gede
+self.lam_coeffs = -coneqs_mat.jacobian(qd) + +#Determining the third term in Lagrange's EOM +#term3 = ((self.lam_vec).transpose() * self.lam_coeffs).transpose() +term3 = self.lam_coeffs.transpose() * self.lam_vec + +#Taking the

Re: [sympy] LagrangesMethod class for sympy.physics.mechanics. (#1460)

2012-08-07 Thread Aaron Meurer
+below_eye = zeros(n+m,n) +A = (self.mass_matrix).col_join((self._m_cd).row_join(zeros(m,m))) +below_I = below_eye.row_join(A) +return I.col_join(below_I) +else: + A = row1.col_join(row2) + return A + +

Re: [sympy] Fixed an error in kinematic differential equations. (#1469)

2012-08-07 Thread Jason Moore
@angadhn FYI: the online docs also have the error: http://docs.sympy.org/dev/modules/physics/mechanics/examples.html#the-rolling-disc --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1469#issuecomment-7574485 -- You received this message because you

Re: [sympy] combsimp fix (#1462)

2012-08-07 Thread Christopher Smith
There's a tiny bit of cleanup in the processing of a run in the code, but that will have to wait. My computer just died and I don't know when it will be fixed. In any case, it's a minor change that can be added later. Since this is passing and Tom is OK with the changes, I will commit this.

Re: [sympy] combsimp fix (#1462)

2012-08-07 Thread Christopher Smith
There's a tiny bit of cleanup in the processing of a run in the code, but that will have to wait. My computer just died and I don't know when it will be fixed. In any case, it's a minor change that can be added later. Since this is passing and Tom is OK with the changes, I will commit this.

Re: [sympy] Fixed an error in kinematic differential equations. (#1469)

2012-08-07 Thread Christopher Smith
@angadhn , will you fix the error in the docs, too? (Just want to make sure this doesn't get forgotten as we move towards the release of 0.7.2). --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1469#issuecomment-7574702 -- You received this message

Re: [sympy] Fixed an error in kinematic differential equations. (#1469)

2012-08-07 Thread Angadh Nanjangud
yup. --- Reply to this email directly or view it on GitHub: https://github.com/sympy/sympy/pull/1469#issuecomment-7574706 -- You received this message because you are subscribed to the Google Groups sympy-patches group. To post to this group, send email to sympy-patches@googlegroups.com. To

Re: [sympy] Autorat (#1470)

2012-08-07 Thread Stefan Krastanov
**SymPy Bot Summary:** :eight_spoked_asterisk: All tests have passed. *Test command:* setup.py test *master hash*: bd49e6a0b3815b22c559f18535c0e48ab6dfec12 *branch hash*: c1aaae19b1503bc5fe6e38de5193edd2c99843f0 **Interpreter 1:** :eight_spoked_asterisk: All tests have passed. *Interpreter:*

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: