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

Made some fixes.... #58

Open
tinkercnc opened this issue Apr 12, 2023 · 3 comments
Open

Made some fixes.... #58

tinkercnc opened this issue Apr 12, 2023 · 3 comments

Comments

@tinkercnc
Copy link

Hi Mario,

I made "some" fixes (page chunk concatenation) and stripped the code down for my needs aso.
The code is here:
https://github.com/tinkercnc/Bluetti_AC300_ESP32
Further infos follows... ;)

@mariolukas
Copy link
Owner

any update here?

@tinkercnc
Copy link
Author

tinkercnc commented May 28, 2023

Hi Mario,
at the moment I switched the repo to private.
I get in touch ASAP.
:)

@tinkercnc
Copy link
Author

tinkercnc commented May 31, 2023

Hi Mario,
now it's back again.
As said this is a very trimmed version of your code.
But I changed the behavior due to the fact that BLE notify packets are max 20 bytes long.
So I assemble the 20 byte chunks to the full requested length. (hint: CMDPending)
Further I changed some params for better wifi <> bluetooth balance.
Hope you can recycle the code. :)
And thanks for your work!
It helps me a lot!

Schoene Gruesse nach D.

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

No branches or pull requests

2 participants