Some overdue updates for Vicon Positioning System's doc#7824
Some overdue updates for Vicon Positioning System's doc#7824MatthewT0809 wants to merge 1 commit into
Conversation
| @@ -1,253 +1,365 @@ | |||
| .. _common-vicon-for-nongps-navigation: | |||
| .. _Vicon Positioning System as fake GPS for Indoor Flight: | |||
There was a problem hiding this comment.
thanks for this. we shouldn't change the first line though, we always make the link the same as the filename prefixed with an underscore
| Using a Vicon indoor positioning system | ||
| ======================================= | ||
| =========================================================== | ||
| Vicon Positioning System as fake GPS for Indoor Flight |
There was a problem hiding this comment.
TBH, I'm not a fan of "fake GPS" in the title so let's just leave it as it was
MatthewT0809
left a comment
There was a problem hiding this comment.
Small changes with the 1st line and the name of the page, as requested.
|
Checking in with this. Any updates on this request? |
|
has merge conflicts and Randy must approve |
Hwurzburg
left a comment
There was a problem hiding this comment.
you must fix issues with copywiki and fix merge conflicts
|
last merge should be a CI failure...merges of master are not permitted...if you MUST (only needed if a master change makes CI fail on the PR)...rebase it |
|
See if that worked, rebased, added blimp, not too sure what you meant with rover's TOC but I updated the doc in /common and I didn't change the naming convention as Rmackay9 suggested, so rover should be able to refer to it. Few fixes with referencing in my guide that I missed which failed the build. I tried the build again locally and no warning from my end. |
|
BTW, I think it was @tridge who wrote this page. I've never actually used a Vicon system. |
|
okay, will gate this on @tridge approval |
Feel free to change the wording and everything as you see fit.
I am reachable via email @ Matthewchtam@gmail.com.