Tom Pittenger
Tom Pittenger
Old PR 30785 mentioned above is closed, this now depends on new PR https://github.com/ArduPilot/ardupilot/pull/31048
@tolesam since the new PR is totally different you should update common-custom-firmware.rst
@Hwurzburg The code PR merged, but I just noticed a closing ` is missing on here. Can you confirm that it's OK and/or needs to be fixed?
Lgtm. Ready to merge @Hwurzburg ?
@Hwurzburg so things in master are intentionally not documented? What if someone is using master branch? I would think a far better method would be to add that this feature...
Just sounds like more work for you :(
As an example, https://ardupilot.org/plane/docs/common-serial-over-DroneCAN.html says version requirements at the top https://ardupilot.org/plane/docs/common-external-ahrs.html mentions 4.7 which isn't even released yet ?!?!?! https://ardupilot.org/plane/docs/common-network.html#common-network first line says 4.5 requirement
Do you wait for 4.7 release or 4.7 beta? Meanwhile we just leave this good documentation pending and don't let anyone see it until a release happens in.. several months...
That PR kinda kills this project. I know it's an improvement but good lord, please update the examples!