SystemInfoPainter: now displays number of satellites with a fix

LSTS / neptus

Neptus, Command and Control Infrastructure

http://www.lsts.pt/toolchain/neptus/

Other

80 stars 69 forks source link

SystemInfoPainter: now displays number of satellites with a fix #75

Closed krisgry closed 6 years ago

krisgry commented 7 years ago

After a request from our pilots, we now display the number of satellites (with fix) in the SystemInfoPainter. I.e. display GpsFix.satellites

paulosousadias commented 6 years ago

Hi Kristoffer, Thanks for the update. I integrated the code on develop branch but with a small modification to be more compact (https://github.com/LSTS/neptus/commit/71ad3f93586d62d1770ae221edec14c387fb2d86). Because for the LAUV console GPS info is not shown, the way you did it would appear a branc line. Hope you agree with the approach. Please comment.

best

krisgry commented 6 years ago

Hi @paulosousadias, Sorry for the slow reply. This message must have fallen down in a crack.... I have now tested your change, and it looks fine. Thanks for asking :)

BR, KG

paulosousadias commented 6 years ago

Ok, so I will close this PR with the fix being done on 71ad3f9.