PlanetaryRobotics / IrisRoverPackage

Complete software package for the Iris Lunar Rover (CMU).
GNU General Public License v3.0
14 stars 2 forks source link

[ROV-54]: WD detailed report debugging additions #73

Closed mschnur closed 2 years ago

mschnur commented 2 years ago

See this comment for an explanation of what I saw while debugging the detailed report message.

The only change in this PR that actually adds or fixes anything is the addition of the check in LanderComms and UART that makes sure we can send the whole message before trying to send any parts of it. The rest is debugging code that I'd like to keep until we nail down the content of the detailed report beyond a shadow of a doubt. I'll probably end up copying some of the print-to-lander stuff that I used in this PR to debug the Herc <-> WD comms spottiness (ROV-28), which is another reason I want to merge these changes in.

Like #72, the eventual destination of this is fprime-fsw/staging, but these changes were made in tandem with the ones in #72 and without the fixes in that PR these changes will result in additional stack-overflow-related issues. As a result I want to treat this PR similar to #72 and merge this into fprime-fsw once #71 and then #72 are merged.