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

Add RS485 Pylontech support #1052

Open
Manos1966 opened this issue Jun 16, 2024 · 10 comments
Open

Add RS485 Pylontech support #1052

Manos1966 opened this issue Jun 16, 2024 · 10 comments
Labels
enhancement New feature or request

Comments

@Manos1966
Copy link

Is your feature request related to a problem? Please describe.

Currently threre is CANbus Support for Pylontech & other Batteries
The information CAN provides is good but limited compared to the information available from the RS485 port:
CANbus Information shown currently:

CANbus Feedback

Information available when using the RS485 port:

01 Master

It is obvious that the RS485 port is giving more information compared to existing CANbus:

  • It handles each battery separately
  • gives information on temperature from every sensor inside the battery
  • shows the Voltage of each cell inside the battery
  • shows the Balancing situation (mV)
  • can be used combined with the CANbus connection

The software module for OpenDTU-OnBattery has been already developped #622
by skippermeister https://github.com/skippermeister/OpenDTU-OnBattery

Describe the solution you'd like

Add RS485 support to the OpenDTUonBattery in order to make the additional information available.

Additional Information: Both CANbus and RS485 are working at the same time = There is no requirement to change the existing programming which uses the SOC% data from the CANbus.

A solution exists already, monitoring multiple Pylontech Batteries:
#622 adding RS485 Pylontech support

https://github.com/skippermeister/OpenDTU-OnBattery

Describe alternatives you've considered

Use a separate ESP32 with the Fork created by skippermeister
https://github.com/skippermeister/OpenDTU-OnBattery
Both OpenDTUs can be online, collecting information from the Pylontech Battery/ies, using the CANbus and RS485 connection at the same time.

Additional context

No response

@Manos1966 Manos1966 added the enhancement New feature or request label Jun 16, 2024
@Matti07
Copy link

Matti07 commented Jun 23, 2024

Hi @Manos1966, I think that's a very, very good suggestion /reminder.
The display of the cell voltages alone is a breakthrough for me.
But who is able and has the time to implement this?
At first glance, the only people I can think of are the usual suspects, in this case @schlimmchen and @skippermeister.
But how do we convince them to install it?
IIRC the Fusion Board 2.X even has RS485.

@schlimmchen
Copy link
Member

This is indeed a good idea. I am not available to implement this for the foreseeable future. I don't own such hardware and these boxes are too expensive to buy for fun. If I could get my hands on a broken one that I can repair or where I can hook up replacement cells, that would at least enable me to test the feature. Doing stuff like this blind is not rewarding. Moreover, I simply have other goals right now, and I am already spending all the time I can possibly spare with this project.

@Tiese
Copy link

Tiese commented Jun 24, 2024

Hm 🤔

Was nützen all diese Informationen, wenn sie dann doch nicht stimmen...
Bildschirmfoto 2024-06-24 um 11 16 28

Oder liege ich da falsch?

@Manos1966
Copy link
Author

@Tiese
Berechnungsfehler schon korrigiert...

03 Master

03 Slave-1

@Tiese
Copy link

Tiese commented Jun 24, 2024

@Manos1966
Ich dachte schon, ich steh irgendwie auf dem Schlauch.

Aber dennoch: "Hat die US5000 wirklich nur 100Ah Kapazität?"

@Manos1966
Copy link
Author

Manos1966 commented Jun 24, 2024

Ich verstehe die Frage nicht @Tiese was sollte sie denn haben??? es ist eine 4,8kWh Batterie

@Tiese
Copy link

Tiese commented Jun 24, 2024

@Manos1966
Ich Dussel, habe die 4,8kWh mit den 100Ah verwechselt.
Sowas passiert mir leider öfter. 🤪

@Matti07
Copy link

Matti07 commented Jun 25, 2024

Moin @Manos1966, sag mal, könntest Du das eventuell in den Hauptzweig integrieren?
C++ ist leider überhaupt nicht meins und als Rentner will ich mir nicht noch eine Sprache ans Bein binden.

@Manos1966
Copy link
Author

Wenn ich das machen könnte @Matti07 dann hätte ich nicht selber die Anfrage gestellt 😢

@skippermeister
Copy link

siehe meinen letzten Kommentar von heute in #622

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

5 participants