Merge pull request #76 from JHay0112/better-sim

More Stable Physics Simulation

Author: JHay0112

jmath/physics/mechanics.py

@@ -193,10 +193,10 @@ def acceleration(self) -> Vector:
         # F = ma -> a = F/m
         return self.force() / self.mass
 
-    def increment_position(self, time: float = 0.1):
+    def update_position(self, time: float = 0.1):
         """
             Increments the position kinematically over a time interval.
-            Also updates velocity.
+            DOES NOT update velocity.
 
             Parameters
             ----------
@@ -205,6 +205,18 @@ def increment_position(self, time: float = 0.1):
                 The time interval to approximate over.
         """
         self.position = kinematic_position(self.position, self.velocity, self.acceleration(), time)
+
+    def update_velocity(self, time: float = 0.1):
+        """
+            Updates the velocity kinematically over a time interval.
+            DOES NOT update position.
+
+            Parameters
+            ----------
+
+            time
+                The time interval to approximate over.
+        """
         self.velocity = kinematic_velocity(self.velocity, self.acceleration(), time)
 
     @__non_zero_distance

jmath/physics/simulation.py

@@ -108,9 +108,13 @@ def mainloop(self, time_interval, *args):
 
             for object in self.objects:
 
-                object.increment_position(time_interval)
+                object.update_velocity(time_interval)
 
-                if self.width < object.position[0] or object.position[0] < 0 or self.height < object.position[1] or object.position[1] < 0:
+            for object in self.objects:
+
+                object.update_position(time_interval)
+
+                if self.width < object.shape.x or object.shape.x < 0 or self.height < object.shape.y or object.shape.y < 0:
                     self.canvas.delete(object.shape.canvas_obj)
                     self.objects.remove(object)
                 else: