Ryan
Ryan
> Is this still open? Yep!
> @Ryanf55 The second last hexadecimal is supposed to be checksum right? Since 0x3 is an ending tag following how binary commands were handled in `AP_GPS_GSOF::requestBaud` but, hexadecimals preceding don't...
> @Ryanf55 I have mostly done the work described in this ticket. But, I have been stuck in implementing the arming/disarming scenarios. I tried scanning through the source code for...
Yea, I thought so too. Did you see this PR? https://github.com/ArduPilot/ardupilot/pull/26143
We had mentioned putting together a python script to deploy this in hardware and then run an integration test. Would it be possible to put something together like that so...
Do you want to rebase and we can try to get this in soon? I now have an approprite hardware test setup with debugger.
> Hi, I read the source code of custom server. I wanted to confirm that just need to add just the appropriate entry in `BUILD_OPTIONS`, or are there changes need...
> AP_DDS_ENABLED Yep! Right here: https://github.com/ArduPilot/ardupilot/blob/8c72304ab912b42d38bd3301fd964aaec7149e77/Tools/ardupilotwaf/boards.py#L85 By default, running `./waf configure --board sitl` does NOT include `--enable-dds`, so the above `if cfg.options.enable_dds` evaluates to False, and DDS is not included.
> The compiler is unable to recognize any definition of AP_DDS_ENABLED, even though it's clearly defined in hwdef.h generated by the custom build sever. Edit: Fixed by adding the missing...
Nice, seems like a lot less code duplication. Were you able to run the size compare?