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

Renogy initial beta #75

Merged
merged 11 commits into from
Dec 8, 2021
Merged

Renogy initial beta #75

merged 11 commits into from
Dec 8, 2021

Conversation

gambleben
Copy link
Contributor

First attempt to get Renogy Smart Lithium battery working with this driver.

Implemented code to read battery serial number, voltage, current, and state of charge.

First attempt to get Renogy Smart Lithium battery working with this driver.
Fixed serial number parsing
Fix python 2.7 errors.
manually set cell count
Fix cell voltages, added cell temps, total capacity, bms temp 1 and 2, cleaned up some code.
Used to test the Renogy battery class without needing to be on a venus os device
Minor tweak to address to use ASCII 0
Copy link
Owner

@Louisvdw Louisvdw left a comment

Choose a reason for hiding this comment

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

What is the purpose of the etc/dbus-serialbattery/Renogy Test Script.py file.
Is it something other users will also use, or is it just a test file?

Didn't mean to include this test script
@gambleben
Copy link
Contributor Author

I had created the Renogy Test Script.py just to allow me to do some testing from my PC. I removed it from the latest commit.

@Louisvdw Louisvdw merged commit c8577c5 into Louisvdw:renogy Dec 8, 2021
Louisvdw added a commit that referenced this pull request Jan 9, 2022

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
* Renogy initial beta (#75)

* Renogy baseline

First attempt to get Renogy Smart Lithium battery working with this driver.
* Fixed serial number parsing
* Fix python 2.7 errors.
* Manually set cell count
* Fix cell voltages, added cell temps, total capacity, bms temp 1 and 2, cleaned up some code.
* Minor tweak to address to use ASCII 0

* Update build with renogy

Co-authored-by: gambleben <93797963+gambleben@users.noreply.github.com>
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.

None yet

2 participants