just added 2 todos

README.md

@@ -87,6 +87,8 @@ Here is a checklist of what to do together with the key remarks.
   files to make clutter and cause confusion.
 - the pyproject.toml actually brakes local install so i'm just keeping it out for now. sweet jesus
   what a mess
+- instead of using a debug argument and ifs for prints,
+  use built-in python logging
 
 #TODO: write instruction for extending the library to new robots
 -------------

TODOS_2024-03-23

 in order of priority
 --------------------
 1. real-time plotting (need to see what's happening as it's happening,
-   this is here primarily to aid in debugging)
----> have a list of what's possible to plot and then select what's selected 
-2. debug dmp end-of-trajectory shennanigans
-3. implement compliantMoveL
-4. fix getMarkerOffset
+   this is here primarily to aid in debugging) - DONE
+---> have a list of what's possible to plot and then select what's selected - TODO make logging smoother and uniform accross code
+2. debug dmp end-of-trajectory shennanigans - DONE (at least mostly)
+3. implement compliantMoveL - DONE
+4. fix getMarkerOffset - DONE
 5. add waypoints to marker pick up
 6. fix anything that looks problematic in drawing
 7. switch to robot_manager.step() to ensure you're not double-computing things
 8. implement cartesian dmp 
 9. faster visualizing (process in a different thread)
-
-

python/README.md

@@ -17,6 +17,8 @@ so not terribly convenient. it will get fixed in a future ur_rtde release tho
   this to work than to re-do the same functionality from scratch. 
   and you won't really be using it too much anyway let's be real. AND ALSO,
   if you ditch your shitty integration for pinocchio's it will be faster.
+- use logging instead of a debug flag (you're reinventing the wheel and it's also ugly)
+- add types to everything, check them with mypy
 
 # installation
 ------------

python/examples/point_impedance_control.py

@@ -255,7 +255,6 @@ def controlLoopPointImpedance(q_init, controller, robot, i, past_data):
 
 
 
-# TODO: implement
 def controlLoopCartesianPointImpedance(Mtool_init, clik_controller, robot, i, past_data):
     breakFlag = False
     # TODO rename this into something less confusing