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.

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

Feature: Double LilyGo! #109

Merged
merged 16 commits into from
Dec 20, 2023
Merged

Feature: Double LilyGo! #109

merged 16 commits into from
Dec 20, 2023

Conversation

dalathegreat
Copy link
Owner

@dalathegreat dalathegreat commented Nov 23, 2023

WIP

What

This PR adds a new feature. You can now have two LilyGo's talk to each other via serial! (Modbus pins)

Why

Some inverters cannot be on same CAN bus as automotive CAN frames (Solax / Foxess). Having a separate CAN channel for the inverter becomes mandatory. Other HW solutions exist like adding more CAN channels, but this PR adds another alternative to that; Two LilyGos working in tandem:
bild

How

The USER_SETTINGS.h file now has two new defines, that when enabled turns on this functionality:

#define SERIAL_LINK_RECEIVER //Enable this line to receive battery data over RS485 pins from another Lilygo (This LilyGo interfaces with inverter)

#define SERIAL_LINK_TRANSMITTER //Enable this line to send battery data over RS485 pins to another Lilygo (This LilyGo interfaces with battery)

A new library has been added to the lib folder for this serial communication protocol. Massive thanks to @mackelec 🙌

Additional fixes

  • Contactor closing allowed (Battery/inverter) parameters now added to all battery types
  • Tesla Amp value now has improved resolution
  • Tesla batteries only allow contactor closing when inverterAllowsContactorClosing == 1 (SOLAX now works with Tesla)

@dalathegreat dalathegreat changed the title Latest fixes from Andy Feature: Double LilyGo! Nov 23, 2023
@dalathegreat dalathegreat marked this pull request as draft November 27, 2023 22:03
@dalathegreat dalathegreat marked this pull request as ready for review December 20, 2023 21:51
@dalathegreat dalathegreat merged commit ed264bb into main Dec 20, 2023
32 checks passed
@dalathegreat dalathegreat deleted the feature/double-lilygo branch December 23, 2023 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants