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

LiTime BMS #59

Draft
wants to merge 10 commits into
base: master
Choose a base branch
from
Draft

LiTime BMS #59

wants to merge 10 commits into from

Conversation

calledit
Copy link

This is a working implementation but still Work in progress.

It till contains some debug code and is in need of more clean up.

@mr-manuel
Copy link
Owner

Please re-read the requirements, since a few of them are missing. The most important is, that the GitHub Actions run in your repository.

Can you explain the changes to the reinstall-local.sh?

@calledit
Copy link
Author

calledit commented Jul 22, 2024

It's work in progress. (meaning I will add more to the PR as the work progresses)
There is an explanation in the commit message.

etc/dbus-serialbattery/bms/litime.py Outdated Show resolved Hide resolved
etc/dbus-serialbattery/bms/litime.py Outdated Show resolved Hide resolved
etc/dbus-serialbattery/bms/litime.py Outdated Show resolved Hide resolved
etc/dbus-serialbattery/bms/litime.py Outdated Show resolved Hide resolved
etc/dbus-serialbattery/bms/litime.py Outdated Show resolved Hide resolved
etc/dbus-serialbattery/bms/litime.py Outdated Show resolved Hide resolved
etc/dbus-serialbattery/bms/litime.py Outdated Show resolved Hide resolved
etc/dbus-serialbattery/reinstall-local.sh Outdated Show resolved Hide resolved
@mr-manuel mr-manuel marked this pull request as draft July 22, 2024 17:24
@mr-manuel
Copy link
Owner

Ok, then I change it to a draft.

@calledit
Copy link
Author

The docs are in a different branch, I will need to make a different pull request to alter those.
Right?

@mr-manuel
Copy link
Owner

mr-manuel commented Jul 27, 2024

Yes, I moved the docs to a different branch and you have to make the PR there.

@mr-manuel
Copy link
Owner

@calledit any updates on this?

@calledit
Copy link
Author

@calledit any updates on this?

I built this for a setup at my summer house. And I won't be at my summer house until next year so I can't test or build until then.

Last I ran the code was back in July/Agust and it was working fine then. (Or bluetooth keeps crashing, but that did not seam to be an issue with my code)

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