Open philipoe opened 7 years ago
I never noticed that ASL_HIGH_LATENCY would not appear in analyze, except when the mavlink messages were not built in properly...
Right now the communication lost timeout is set to 60s, while we are sending every 40s. So the rate of 1.5 is already there. It was probably a problem only when we both were connected to the iridium server and stole each other every second message. Of course this value can be set to whatever we want with Vehicle::setConnectionLostVariable('timeout in milliseconds').
The analyze widget showed stuff correctly last time we checked.
I will work on fixing the other points once the batteries are soldered.
How is this progressing?
i pushed the branch fix/energybudget (to be tested in flight) where i fixed points 2 and 6.
will take a look at point 3 next.
i can take a look at point 4 as well but i am not familiar with tcp or udp
@acfloria Just left this issue open here because the "receive satcom on two different clients" issue is still open.
Small bugs observed while testing SatCom. This is just quick-notes made together with @mhugentobler :
[x] Analyze widget needs to get HighLatency-messages too. Until now, upon reception of a SatCom message, QGC changes only the map and primary flight display views/widgets. However, the Analyze widget does not receive these values. This needs to be changed (i.e. the ASL High Latency Message values need to be shown there). Note: Most probably, this issue was only due to flawed/too short testing. Keep an eye on this however.
[x] Energy Widget budget needs to be resized once after launching it. This is a bug. Probably, we just need to issue something like "Draw()" once after the Analyze widget is opened. Check this.
[x] energy widget update -> does not update when open before first message arrives -> fix! Note that the energy widget
[ ] Two clients connected to the server via udp2rabbit "steal" satcom-messages from each other, i.e. if client nr. 1 receives a certain message then client nr. 2 will not receive that respective message.
[x] Increase "Communication lost" warning to ca. 1.5x satcom message sending period (if not already done so). This would mean that if we don't loose satcom messages (and their delay is constant), then we should never get the "communication lost" warning.
[x] Energy Widget shows weird values for quantities that are not received via SatCom -> Set these to N/A.