Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improvement: mqtt library replacement #768

Merged
merged 13 commits into from
Jan 19, 2025
Merged

Improvement: mqtt library replacement #768

Show file tree
Hide file tree
Changes from 9 commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
2d97b63
Adding ability to remotely trigger a BMS reset via MQTT
No-Signal Jan 3, 2025
ba37177
Replacing non maintained mqtt library with ESP-MQTT
No-Signal Jan 7, 2025
4c90898
Restructuring code to remove duplicates and reduce flash size
No-Signal Jan 9, 2025
aa7d34a
Fixes to HA auto discovery and adding additional commands
No-Signal Jan 12, 2025
602242a
Improvements for dual battery configurations
No-Signal Jan 14, 2025
3124656
Fixing battery 2 unique_id
No-Signal Jan 15, 2025
89ef408
Updating README file with instructions to update Partition Scheme
No-Signal Jan 15, 2025
afb6cb6
Fixing README file
No-Signal Jan 15, 2025
4085b35
Removing old library from readme file
No-Signal Jan 15, 2025
ad80874
Adding LWT and configuring HA availability to use it and adjusting co…
No-Signal Jan 17, 2025
b1242bc
Updating template MQTT username and password to empty strings to avoi…
No-Signal Jan 19, 2025
0444b4b
Merge branch 'main' into feature/mqtt_library_replacement
No-Signal Jan 19, 2025
fc3f5b4
Merge branch 'main' into feature/mqtt_library_replacement
No-Signal Jan 19, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 7 additions & 6 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,17 +52,19 @@ For more examples showing wiring, see each battery types own Wiki page. For inst

![bild](https://github.com/dalathegreat/Battery-Emulator/assets/26695010/6a2414b1-f2ca-4746-8e8d-9afd78bd9252)

5. The Arduino board should be set to `ESP32 Dev Module` (under `Tools` -> `Board` -> `ESP32 Arduino`) with the following settings:
![alt text](https://github.com/Xinyuan-LilyGO/T-CAN485/blob/main/img/arduino_setting.png)
6. Select which battery type you will use, along with other optional settings. This is done in the `USER_SETTINGS.h` file.
7. Copy the `USER_SECRETS.TEMPLATE.h` file to `USER_SECRETS.h` and update connectivity settings inside this file.
8. Press `Verify` and `Upload` to send the sketch to the board.
5. The Arduino board should be set to `ESP32 Dev Module` and `Partition Scheme` to `Minimal SPIFFS (1.9MB APP with OTA/190KB SPIFFS)` (under `Tools` -> `Board` -> `ESP32 Arduino`) with the following settings:
![alt text](https://github.com/dalathegreat/Battery-Emulator/blob/img/ArduinoSettings.png)
7. Select which battery type you will use, along with other optional settings. This is done in the `USER_SETTINGS.h` file.
8. Copy the `USER_SECRETS.TEMPLATE.h` file to `USER_SECRETS.h` and update connectivity settings inside this file.
9. Press `Verify` and `Upload` to send the sketch to the board.
NOTE: In some cases, the LilyGo must be powered through the main power connector instead of USB-C
when performing the initial firmware upload.
NOTE: On Mac, the following USB driver may need to be installed: https://github.com/WCHSoftGroup/ch34xser_macos

NOTE: If you see garbled messages on the serial console, change the serial console to match the baud rate to the code, currently 115200.

NOTE: If you see the error `Sketch too big` then check you set the Partition Scheme above correctly.

This video explains all the above mentioned steps:
<https://youtu.be/_mH2AjnAjDk>

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work! Could your remove the mention of lib dependency, essentially remove this row from README.md

Expand All @@ -83,7 +85,6 @@ This code uses the following excellent libraries:
- [ayushsharma82/ElegantOTA](https://github.com/ayushsharma82/ElegantOTA) AGPL-3.0 license
- [bblanchon/ArduinoJson](https://github.com/bblanchon/ArduinoJson) MIT-License
- [eModbus/eModbus](https://github.com/eModbus/eModbus) MIT-License
- [knolleary/pubsubclient](https://github.com/knolleary/pubsubclient) MIT-License
- [mackelec/SerialDataLink](https://github.com/mackelec/SerialDataLink)
- [mathieucarbou/AsyncTCPsock](https://github.com/mathieucarbou/AsyncTCPSock) LGPL-3.0 license
- [mathieucarbou/ESPAsyncWebServer](https://github.com/mathieucarbou/ESPAsyncWebServer) LGPL-3.0 license
Expand Down
442 changes: 202 additions & 240 deletions Software/src/devboard/mqtt/mqtt.cpp
View file
Open in desktop

Large diffs are not rendered by default.

Loading
Loading