Radio Control Support #13
Conversation
|
Looks pretty good, I need to do a few minor cleanups and then it should be good to merge 👍 |
| @@ -62,6 +62,8 @@ void Seaking::arm() | |||
| { | |||
| disable(); | |||
| delay(500); | |||
| disable(); | |||
| delay(500); | |||
There was a problem hiding this comment.
Why do we disable twice here? We could just wait 1s after the first disable, right?
Or is this a typo?
There was a problem hiding this comment.
It's a typo. I removed it before, but it must have got back in somehow.
|
I am going to attempt to use an SBUS based RC receiver. This digital format may solve the vague PWM range issues and provide up to 16 channels on a single RX wire. The receiver and transmitter are a bit more expensive, but I think the system should be improved and will allow control of multiple boats (though only one at a time) from a single transmitter. If it does work out, we'll need to decide if we want to merge this PWM based code or the SBUS version. There may be a way of creating a more general class so that both can be supported. I will look into it. |
Adding support for Radio Control