So the docs commit came into Lagrange because I thought that the Lagrange class
was complete w.r.t. the tests; they are identical in number to those in Kane
(apart from that I will also be adding the rolling disc). The two degree of
freedom test in my case is the double pendulum. Like Kane, I
I think this error may be due to the whole rebasing issue. I have fixed this in
my last commit. Sorry for the confusion.
---
Reply to this email directly or view it on GitHub:
https://github.com/sympy/sympy/pull/1407#issuecomment-7606048
--
You received this message because you are subscribed
@certik I am uncertain of how to interpret the docstring error but I have
corrected what I believe was causing the error. I believe I have addressed all
the concerns.
---
Reply to this email directly or view it on GitHub:
https://github.com/sympy/sympy/pull/1407#issuecomment-7607337
--
You
m*r*v*N.z
return self.point.pos_from(point) ^ (self.mass *
self.point.vel(frame))
+
+def kinetic_energy(self, frame):
+Kinetic energy of the particle
+
+The kinetic energy, T, of a particle,P, is given by
+
+T = 0.5 * m *
still nothing. but it looks like the tests passed.
---
Reply to this email directly or view it on GitHub:
https://github.com/sympy/sympy/pull/1407#issuecomment-7633637
--
You received this message because you are subscribed to the Google Groups
sympy-patches group.
To post to this group, send
So it appears that I have mistakenly rebased a couple of my branches and so my
commits (including the most recent one on this page) aren't showing up
correctly. Any way I can fix this?
---
Reply to this email directly or view it on GitHub:
Actually, it's fine here. Never mind.
---
Reply to this email directly or view it on GitHub:
https://github.com/sympy/sympy/pull/1460#issuecomment-7575781
--
You received this message because you are subscribed to the Google Groups
sympy-patches group.
To post to this group, send email to
+# 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
if not isinstance(bodylist, list):
raise TypeError('System elements must be supplied as a list')
else:
-angularmomentum_sys = 0
+angular_momentum_sys = 0
done.
---
Reply to this email directly or view it on GitHub:
I have also fixed the html doc errors in the latest commit and addressed all
issues brought up thus far.
---
Reply to this email directly or view it on GitHub:
https://github.com/sympy/sympy/pull/1407#issuecomment-7581049
--
You received this message because you are subscribed to the Google
+ 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')
+
10*N.z
+bodylist = list(bodylist)
fixed this. I didn't realise this was in the momenta functions too.
---
Reply to this email directly or view it on GitHub:
https://github.com/sympy/sympy/pull/1407/files#r1336833
--
You received this message because you are subscribed
10*N.x + 500*N.y
+bodylist = list(bodylist)
if not isinstance(bodylist, list):
same as below. addressed now.
---
Reply to this email directly or view it on GitHub:
https://github.com/sympy/sympy/pull/1407/files#r1336838
--
You received this message because you are
+
+Examples
+
+
+ from sympy.physics.mechanics import Particle, Point
+ from sympy import symbols
+ m, g, h = symbols('m g h')
+ O = Point('O')
+ P = Particle('P', O, m)
+ P.set_potential_energy(m * g * h)
+
+
@certik I have stopped rebasing. Sorry about that mess.
---
Reply to this email directly or view it on GitHub:
https://github.com/sympy/sympy/pull/1407#issuecomment-7597305
--
You received this message because you are subscribed to the Google Groups
sympy-patches group.
To post to this group,
@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
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
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
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
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
The error message for the 'coords' method has been corrected.
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/1466
-- Commit Summary --
* Corrected
21 matches
Mail list logo