hexy icon indicating copy to clipboard operation
hexy copied to clipboard

Move servo setup to a config file.

Open robottwo opened this issue 7 years ago • 5 comments

This patch moves the controller addresses to a config file, along with the servo PWM range of motion.

This allows one to use any number of servo controllers on the i2c bus, and also makes it easier to customize the servo calibration.

This patch also includes a Servo Calibration GUI, which makes it much easier to calibrate both the controller and channel, as well as the min and max pwm for each servo.

robottwo avatar Apr 02 '17 00:04 robottwo

@robottwo

Do you have a demo video showcasing sample usage of your changes (like the gui and maybe a sample case on using more than two servo controllers (assuming it's 16 servos per controller wow that's a lot of servos!) Just curious, did you run this on a raspberry pi zero as well? Thanks in advance.

mithi avatar Apr 06 '17 23:04 mithi

Nope to all 3. But I did find it useful just when connecting to two controllers, since different people may put them on different addresses, and put the servos on different channels.

-Dan

On Apr 6, 2017, at 7:01 PM, Mithi Sevilla [email protected] wrote:

@robottwo

Do you have a demo video showcasing sample usage of your changes (like the gui and maybe a sample case on using more than two servo controllers (assuming it's 16 servos per controller wow that's a lot of servos!) Just curious, did you run this on a raspberry pi zero as well? Thanks in advance.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

robottwo avatar Apr 07 '17 00:04 robottwo

@robottwo

Thanks for your reply. I do understand how extremely useful this patch is, I'd merge this but I'd have to test it on my hardware setup first and maybe do a short readme. Thanks!

mithi avatar Apr 07 '17 01:04 mithi

I updated the readme to talk about the new functionality.

robottwo avatar Apr 08 '17 14:04 robottwo

@robottwo Thanks! I'll test this on my hexapod this week and if I don't run into any issues, I'd merge this anytime this week! Thanks again for the readme!

mithi avatar Apr 13 '17 04:04 mithi