krzysztofkuczek / QCZEK_LRS

RC LRS 1W system, based on LORA RF modules.
17 stars 5 forks source link

no of satelites not displayed with UBLOX temeletry #6

Open krzysztofkuczek opened 5 years ago

krzysztofkuczek commented 5 years ago

needs to be confirmed if user enabled ublox msg MSG_SOL

giacomo892 commented 5 years ago

Can you just make the UBLOX telemetry work only if NAV_PVT message is enabled?

krzysztofkuczek commented 5 years ago

but why ? :)

giacomo892 commented 5 years ago

Simpler approach. Just one message to decode to have all the data you need. Modern FC (at 10Hz) just enable NAV_PVT on the module, so if you want to tap the GPS TX wire to connect it to QLRS you are already out of luck

krzysztofkuczek commented 5 years ago

well, well, but this message is 90 bytes long, and i don't have enough ram for processing. this change requires testing, to make sure that heap will not be overflowed ... will do that, but not now. sorry

giacomo892 commented 5 years ago

don't worry, my was just an improvement suggestion :)