-
Notifications
You must be signed in to change notification settings - Fork 145
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
Poller fix #53
base: master
Are you sure you want to change the base?
Poller fix #53
Conversation
them for each call.
- reduce unnecessary wrapper processes
…s and removes the need to provide qpiri/qpiws/qmod/qpigs reply sizes in the config. - also fixed some compiler warnings.
Parametr “Battery_redischarge_voltage” set in “:-0.0,” while the QPIRI value displays the desired parameter: Tue Oct 12 11:16:40 2021 INVERTER: QPIRI: 98 bytes read: (220.0 13.6 220.0 50.0 13.6 3000 2400 24.0 23.5 21.0 28.2 27.0 0 10 060 0 2 3 1 01 0 0 25.0 0 1` |
Hi @mstsclev, this version still has a bug in it. Can you try the latest version from the master branch of my fork? https://github.com/kchiem/docker-voltronic-homeassistant/tree/master/sources/inverter-cli |
Hello, I can't find the right combination of buffer sizes for my MKS SE II 5kW. If I understand, there is no need for those anymore in this fork. If I only need the sensors to show up in HA for now, do you think this fork would help me? @kchiem Thanks for your work! |
Hi, yes, try checking out the inverter-cli from the link above your comment:
$ cmake . && make
$ ./inverter_poller -1 |
Just an FYI - manually pulled these changes in, and works great! Still thinking about re-sending charge modes when the mode doesn't actually change (sometimes either the inverter doesn't change modes, or doesn't get the raw command if polling at the same time) |
@ned-kelly Could you please check out this pr? I struggled a bit trying to set the max charge rate before learning about the limitation there is on USB. Thanks! |
Hello, what am I doing wrong? does not work... My inverter: MPP Solar / Infinisolar MPI 10k hybrid inverter root@raspberrypi:/opt/ha-inverter-mqtt-agent/sources/inverter-cli# cmake . && make inverter.conf device=/dev/hidraw0 run_interval=120 amperage_factor=1.0 watt_factor=1.01 qpiri=97 root@raspberrypi:/opt/ha-inverter-mqtt-agent/sources/inverter-cli# ./inverter_poller -1 -d |
@nrm21's changes:
to the poller binary that fixes commands longer than 5 characters.
it also allows it to take arbitrary length responses, doing away with the need to configure response lengths for QPIRI/QPIWS/QMOD/QPIGS commands in the inverter.conf file.
I also fixed a couple of compiler warnings.
Unfortunately, I couldn't figure out how to isolate the pull request to just the poller's files, so some other minor changes I made to the mqtt shell scripts are included.