edgetx icon indicating copy to clipboard operation
edgetx copied to clipboard

Add GPS Option

Open HThuren opened this issue 2 years ago • 10 comments

Minor change, add GPS option for x12 and language da in .py yfiles

HThuren avatar Sep 11 '22 10:09 HThuren

Raphael is going to look into this - as for the X12 most of this should not be needed - it has a GPS as standard, so should simply be ON lower down in the code, and also not a configurable option in the UI since the UART/AUX is already claimed by the GPS.

pfeerick avatar Sep 12 '22 09:09 pfeerick

Thats good :-)

HThuren avatar Sep 12 '22 17:09 HThuren

The GPS on the x12 has a dedicated connector. Its the only physical connector on that Radio where a User can connect other stuff without soldering. (ESP Modules and more)

(I would share a Photo, but im in the Hospital still since 2 Months now)

I like that idea, because its User Friendly.😉 +1! But i also like the Idea for BT as a Option but without explicit use the Compile option, set to ON as Standard And dBluetooth=OFF as an Option if someone needs it. The Horus BT Module is not really compatible with android or other stuff and a Solid Data Connection is just not possible. Not even the different Frsky Modules are compatible between each other. 🤣 Because of that, i removed it and connected a hc05 wich is way more reliable.

brainbubblersbest avatar Sep 21 '22 16:09 brainbubblersbest

Good Health to you :-)

HThuren avatar Sep 21 '22 21:09 HThuren

Good Health to you :-)

Thank you. I hope to get out soon. Still have something to test here and already feel guilty for the huge delay. 😅

brainbubblersbest avatar Sep 22 '22 07:09 brainbubblersbest

Thank you. I hope to get out soon. Still have something to test here and already feel guilty for the huge delay. 😅

Don't stress over it... your health is more important! Just glad to hear you're still alive and kicking! 😉

pfeerick avatar Sep 22 '22 07:09 pfeerick

Minor change, add GPS option for x12 and language da in .py yfiles

You add the Bluetooth Compile Option for x10express? Is that right? Dont you want that for the x12?

brainbubblersbest avatar Sep 22 '22 15:09 brainbubblersbest

Yes, thats was confused, schuld be X12

HThuren avatar Sep 22 '22 19:09 HThuren

Does gps=off or/and bluetooth=off enable the aux_serials1/2 instead? I think just these changes are not enough here.

This might be fyi serial refactor PR

brainbubblersbest avatar Sep 23 '22 01:09 brainbubblersbest

Your memory does not deceive you ;)

Configuring BT serial on run-time is coming next.

GPS supported in the new serial framework, so can be on any available serial/aux port, but bluetooth isn't, and will just remove the option for the serial port that the code expects the Bluetooth to be at (i.e. because it has a dedicated connector).

pfeerick avatar Sep 23 '22 02:09 pfeerick

Hi @pfeerick , do you want me to close this PR ?

HThuren avatar Oct 18 '22 21:10 HThuren

I think so, yes. We did establish this wasn't going to work this way, so no point leaving it open now.

pfeerick avatar Oct 19 '22 02:10 pfeerick