search

richonguzman / LoRa_APRS_Tracker

LoRa APRS Tracker with Tx and Rx capabilities, Messages, Wx, Winlink and more...
MIT License
169 stars 46 forks source link

Unable to compile tracker for TTGO T-Beam v0.7 #19

Closed btor closed 10 months ago

btor commented 11 months ago

Definitions for Power saving mode missing.

Compiling .pio\build\ttgo-t-beam-v0_7\src\station_utils.cpp.o In file included from src/LoRa_APRS_Tracker.cpp:16: src/power_utils.h:16:14: error: 'TwoWire' has not been declared bool begin(TwoWire &port); ^~~ In file included from src/power_utils.cpp:1: src/power_utils.h:16:14: error: 'TwoWire' has not been declared bool begin(TwoWire &port); ^~~ src/power_utils.cpp:10:29: error: 'bool PowerManagement::begin' is not a static data member of 'class PowerManagement' bool PowerManagement::begin(TwoWire &port) { ^~~ src/power_utils.cpp:10:29: error: 'TwoWire' was not declared in this scope src/power_utils.cpp:10:38: error: 'port' was not declared in this scope bool PowerManagement::begin(TwoWire &port) { ^~~~ src/power_utils.cpp:10:38: note: suggested alternative: 'word' bool PowerManagement::begin(TwoWire &port) { ^~~~ word src/power_utils.cpp: In member function 'double PowerManagement::getBatteryVoltage()': src/power_utils.cpp:138:1: error: no return statement in function returning non-void [-Werror=return-type]
} ^ src/power_utils.cpp: In member function 'double PowerManagement::getBatteryChargeDischargeCurrent()':
src/power_utils.cpp:151:1: error: no return statement in function returning non-void [-Werror=return-type]
} ^ src/power_utils.cpp: In member function 'bool PowerManagement::isBatteryConnected()': src/power_utils.cpp:160:1: error: no return statement in function returning non-void [-Werror=return-type]
} ^ src/power_utils.cpp: In member function 'bool PowerManagement::isChargeing()': src/power_utils.cpp:169:1: error: no return statement in function returning non-void [-Werror=return-type] } ^ cc1plus.exe: all warnings being treated as errors In file included from src/menu_utils.cpp:7: src/power_utils.h:16:14: error: 'TwoWire' has not been declared bool begin(TwoWire &port); ^~~ In file included from src/station_utils.cpp:5: src/power_utils.h:16:14: error: 'TwoWire' has not been declared bool begin(TwoWire &port); ^~~ Compiling .pio\build\ttgo-t-beam-v0_7\libed7\TinyGPSPlus\TinyGPS++.cpp.o Compiling .pio\build\ttgo-t-beam-v0_7\lib54d\Time\DateStrings.cpp.o Compiling .pio\build\ttgo-t-beam-v0_7\lib54d\Time\Time.cpp.o Compiling .pio\build\ttgo-t-beam-v0_7\lib3f2\OneButton\OneButton.cpp.o [.pio\build\ttgo-t-beam-v0_7\src\LoRa_APRS_Tracker.cpp.o] Error 1 [.pio\build\ttgo-t-beam-v0_7\src\power_utils.cpp.o] Error 1 [.pio\build\ttgo-t-beam-v0_7\src\menu_utils.cpp.o] Error 1 [.pio\build\ttgo-t-beam-v0_7\src\station_utils.cpp.o] Error 1 ======================================== [FAILED] Took 47.59 seconds ========================================

Environment Status Duration

ttgo-t-beam-v0_7 FAILED 00:00:47.590 =================================== 1 failed, 0 succeeded in 00:00:47.590

richonguzman commented 11 months ago

Hi,

are you choosing specially V0_7 on VSCODE and importing all through Platformio?

btor commented 11 months ago

Hi, Thanks for the fast response. Yes, I use VSCODE with Platformio and V0_7 is selected before compilation. Power saving is not implemented in power_utils.cpp for V0_7 Best Regards, Bela

btor commented 11 months ago

Hello Ricardo,

I added necessary definitions to power_utils.h and power_utils.cpp required for a succesful compilation. Since I have no information on V0_7 hardware, no power manadgement related code was added. Included you'll find the zipped files.

Best regards, Bela

power_utils.zip

richonguzman commented 11 months ago

this is added now to the next firmware available and compiles good, so it should work now, thanks a lot

richonguzman commented 11 months ago

should the V0_7 be called on the powermanagment::setup() also?

btor commented 11 months ago

No there is nothing to initialize in setup() I added battery voltage measurement to power_utils.cpp and menu_utils.cpp and comments were added regarding v0.7 to power_utils.cpp, power_utils.h Included you'll find the mdified files. src.zip

richonguzman commented 11 months ago

could you confirm it works for you with this mods?

btor commented 11 months ago

Yes, it works, I tested the mods with a V0.7 board. Best regards, Bela / HB9HEY

richonguzman commented 11 months ago

I uploaded a new firmware , could you please confirm it still runs well on yours?

Saludos,

Ricardo / Richon (Enviado desde mi celular a tubos)

El 10-08-2023, a las 15:15, Béla Török @.***> escribió:



Yes, it works, I tested the mods with a V0.7 board. Best regards, Bela / HB9HEY

— Reply to this email directly, view it on GitHubhttps://github.com/richonguzman/LoRa_APRS_Tracker/issues/19#issuecomment-1673772954, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQWBUGXV3QX254LCBUMTHQ3XUUXOPANCNFSM6AAAAAA3DB2DW4. You are receiving this because you commented.Message ID: @.***>

btor commented 11 months ago

I can't compile the new firmware for v0_7. I get the following compiler error:

Compiling .pio\build\ttgo-t-beam-v0_7\src\lora_utils.cpp.o src/lora_utils.cpp:1:10: fatal error: RadioLib.h: No such file or directory

Environment Status Duration

ttgo-t-beam-v0_7 FAILED 00:00:19.759 ================================ 1 failed, 0 succeeded in 00:00:19.759 ================================

richonguzman commented 11 months ago

Radiolib is only for sx1268 board .. so it should not be a problem with yours (v0_7)

Try deleting it at your PC on the lora_utils.cpp and platformio ini to see how it goes

Saludos,

Ricardo / Richon (Enviado desde mi celular a tubos)

El 11-08-2023, a las 08:31, Béla Török @.***> escribió:



I can't compile the new firmware for v0_7. I get the following compiler error:

Compiling .pio\build\ttgo-t-beam-v0_7\src\lora_utils.cpp.o src/lora_utils.cpp:1:10: fatal error: RadioLib.h: No such file or directory

include

^~~~ compilation terminated. Archiving .pio\build\ttgo-t-beam-v0_7\lib04d\libWire.a Compiling .pio\build\ttgo-t-beam-v0_7\libaa6\SPI\SPI.cpp.o Compiling .pio\build\ttgo-t-beam-v0_7\libbd1\Adafruit BusIO\Adafruit_BusIO_Register.cpp.o Compiling .pio\build\ttgo-t-beam-v0_7\libbd1\Adafruit BusIO\Adafruit_I2CDevice.cpp.o Compiling .pio\build\ttgo-t-beam-v0_7\libbd1\Adafruit BusIO\Adafruit_SPIDevice.cpp.o Compiling .pio\build\ttgo-t-beam-v0_7\lib078\Adafruit GFX Library\Adafruit_GFX.cpp.o Compiling .pio\build\ttgo-t-beam-v0_7\lib078\Adafruit GFX Library\Adafruit_GrayOLED.cpp.o *** [.pio\build\ttgo-t-beam-v0_7\src\lora_utils.cpp.o] Error 1 ===================================== [FAILED] Took 19.76 seconds =====================================

Environment Status Duration

ttgo-t-beam-v0_7 FAILED 00:00:19.759 ================================ 1 failed, 0 succeeded in 00:00:19.759 ================================

— Reply to this email directly, view it on GitHubhttps://github.com/richonguzman/LoRa_APRS_Tracker/issues/19#issuecomment-1674690468, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQWBUGV6NRV7JQ2S22XJSDLXUYQYLANCNFSM6AAAAAA3DB2DW4. You are receiving this because you commented.Message ID: @.***>

btor commented 11 months ago

After adding defined(TTGO_T_Beam_V0_7) in multiple paces in lora_utils.cpp and making "full clean" in Visual Studio Code I could compile the current firmware for V0.7

The modifications I submitted yesterday -> display the Battery voltage on the display were not merged to this firmware. Should I implement it?

Best Regards, Bela / HB9HEY

lora_utils.zip

richonguzman commented 11 months ago

I will do both today(night) and message you to test ;)

Saludos,

Ricardo / Richon (Enviado desde mi celular a tubos)

El 11-08-2023, a las 09:36, Béla Török @.***> escribió:



After adding defined(TTGO_T_Beam_V0_7) in multiple paces in lora_utils.cpp and making "full clean" in Visual Studio Code I could compile the current firmware for V0.7

The modifications I submitted yesterday -> display the Battery voltage on the display were not merged to this firmware. Should I implement it?

Best Regards, Bela / HB9HEY

lora_utils.ziphttps://github.com/richonguzman/LoRa_APRS_Tracker/files/12321934/lora_utils.zip

— Reply to this email directly, view it on GitHubhttps://github.com/richonguzman/LoRa_APRS_Tracker/issues/19#issuecomment-1674806630, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQWBUGUMQXBNUA5BJLIMQL3XUYYN3ANCNFSM6AAAAAA3DB2DW4. You are receiving this because you commented.Message ID: @.***>

richonguzman commented 10 months ago

it seems all was added correctly, please test the "new" Main 2023.08.12 firmware to validate it and share your callsign to add this to readme as to share your help ;)

btor commented 10 months ago

I could compile the firmware for V0_7. Since I am currently abroad, I could not test the functionality with an igate. I'll test it next week when I am back home again.

I noticed a missing word in power_utils.h the following section in the top of the file:

ifdef TTGO_T_Beam_V0_7

// The V0.7 boards have power managment components connected to TwoWire.

Please add a "no" between "have" and "power" like this:

ifdef TTGO_T_Beam_V0_7

// The V0.7 boards have no power management components connected to TwoWire. //

The button does not work properly with my V0_7 board, it is almost impossible to navigate in and between the menus. Perhaps the button does not work properly, unfortunately, I can't test it at the moment. I'll test it wen I'm back home again.

Best Regards, Bela HB9HEY

btor commented 10 months ago

I solved the button issue with V0_7 board.. BUTTON_PIN was defined as 38 in pins_config.h, but the button is connected to pin 39 on V0_7 board

pins should be defined for the V0_7 board as follows:

ifdef TTGO_T_Beam_V0_7

define GPS_RX 15

define GPS_TX 12

define BUTTON_PIN 39

endif

After changing the button definition the menu works as expected on my V0_7 board.

Kind regard, Bela HB9HEY

richonguzman commented 10 months ago

added the corrections

btor commented 10 months ago

Last night I tested the tracking functionality, it works. You can visualize the result with aprs.fi callsign HB9HEY-7

richonguzman commented 10 months ago

Great, so it works and even the info on screen and the middle button works

Please confirm to close this issue

btor commented 10 months ago

You can close this issue. Thanks for the excellent support. Kind regards, Bela