MyRobotLab / myrobotlab

Open Source Java Framework for Robotics and Creative Machine Control
Apache License 2.0
226 stars 108 forks source link

Servo calibration #164

Closed moz4r closed 6 years ago

moz4r commented 6 years ago

Today if we want to calibrate a servo ( ex. for inmoov ), it's not clear enough for everyone.

I think we can use existing servo UI and use existing json related to store values. ( and it's almost worky ) . Better for real-time adjustments.

hairygael commented 6 years ago

Yes this needs to be more clear. Too many users fail or do not understand how to calibrate a servo to report the min and max values in their config files. In the end they keep the default values that are preset in the config files and might burn their servo.

Gael Langevin Creator of InMoov InMoov Robot http://www.inmoov.fr @inmoov http://twitter.com/inmoov

2018-02-27 22:13 GMT+01:00 Anthony notifications@github.com:

Today if we want to calibrate a servo ( ex. for inmoov ), it's not clear enough for everyone.

  • Launch servo service
  • Connect to controller
  • Play with cursor
  • Store limits inside config file / diy script

I think we can use existing servo UI and use existing json related to store values. ( and it's almost worky ) . Better for real-time adjustments.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/MyRobotLab/myrobotlab/issues/164, or mute the thread https://github.com/notifications/unsubscribe-auth/AIF2x20f4y9Z-m-kxbtpsvVKkc9fxGK6ks5tZG_-gaJpZM4SVpzz .

kwatters commented 6 years ago

I suggest moving all of this calibration into java land. We should add a button to the InMoov for "load" and "save" calibration. This way users can use the GUI , set their min/max , com ports, velocity.. etc.. and then click save . No more human editing of files.

moz4r commented 6 years ago

what ! No more human editing ? I like notepad++ :) yep agree... I think of 2 buttons save ( not really a button this is/will be automatic ) -> store stuff inside service related json at shutdown based on py commands or gui changes. But maybe it will be safer to add a save button. export -> this will create a thing to implement inside a diy script ( this is not mandatory because diy script will automatic load previous settings stored inside servoX.json ( if there is no override commands ), so useful to override every time )

hairygael commented 6 years ago

The issue for beginners is not the "saving" but to understand what is the goal of calibration and how to proceed. Sebastien had done a very nice graphical proposition to add to the InMoov site for beginners, but we kind of stopped because it would be better to implement such graphic calibrator directly on MyRobotLab.

Have a look at what EZ has done ( using my picture without permission, huh!) https://www.youtube.com/watch?v=ECWUPJ0oDaQ&t=26s

Could we do something similar since i have the permissions to use my pictures, ehehe

Gael Langevin Creator of InMoov InMoov Robot http://www.inmoov.fr @inmoov http://twitter.com/inmoov

2018-03-06 22:41 GMT+01:00 Kevin Watters notifications@github.com:

I suggest moving all of this calibration into java land. We should add a button to the InMoov for "load" and "save" calibration. This way users can use the GUI , set their min/max , com ports, velocity.. etc.. and then click save . No more human editing of files.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/MyRobotLab/myrobotlab/issues/164#issuecomment-370938512, or mute the thread https://github.com/notifications/unsubscribe-auth/AIF2x2s0fxuEQANt7a85yUOiv5g4PjTGks5tbwKHgaJpZM4SVpzz .

moz4r commented 6 years ago

The save/export calibrate will remove the actual text edit step. Just 1 thing less to do. I think we have almost all pieces for fireworks, the core of servo calibration process will be inside servo gui screen. Maybe we can polish more servo Gui . Ideas welcome like 2 tabs inside servo GUI ( basics / advanced )... WIP to do similar thing inside i02 ( swing & layout things can cause headaches, example fit all screen size with a picture like this ) . We will implement something closer, will post soon a proposition to talk about. not yet arrived at the skeleton part :) image

hairygael commented 6 years ago

Woo, that seems to be nice and promising!

Gael Langevin Creator of InMoov InMoov Robot http://www.inmoov.fr @inmoov http://twitter.com/inmoov

2018-03-06 23:47 GMT+01:00 Anthony notifications@github.com:

The save/export calibrate will remove the actual text edit step. Just 1 thing less to do. I think we have almost all pieces for fireworks, the core of servo calibration process will be inside servo gui screen. Maybe we can polish more servo Gui . Ideas welcome like 2 tabs inside servo GUI ( basics / advanced )... WIP to do similar thing inside i02 ( swing & layout things can cause headaches, example fit all screen size with a picture like this ) . We will implement something closer, will post soon a proposition to talk about. not yet arrived at the skeleton part :) [image: image] https://user-images.githubusercontent.com/18124594/37062870-fcc595e6-2197-11e8-9cd1-c21987f082f8.png

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/MyRobotLab/myrobotlab/issues/164#issuecomment-370957033, or mute the thread https://github.com/notifications/unsubscribe-auth/AIF2xxpq4hmPNLNyMPhIABxXXYCIynvgks5tbxIRgaJpZM4SVpzz .

moz4r commented 6 years ago

cooked ! http://myrobotlab.org/content/servo-calibration This is generic calibration process inside mrl , a sub part for next works for inmoov