Combined #86
Open
Combined #86
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Accept the QPI, QID, QVFW, QVFW2, QFLAG, QBOOT, and QOPM commands.
The HS_MS_MSX protocol specification in the repository says for the QPIRI command value R is “Parallel max num”. On my system R returns 9, while the source code expected "-" in scanf(). This change alters the input of scanf() to accept any character in the position for R. Closes ned-kelly#71
When many inverter_poller at the same time send commands, the result is undefined.
This locks exclusively the configuration file, while the process is running,
so that simultaneous instances have to wait until the file is released.
For some reason locking the device /dev/hidraw0 does not work for me,
I get sometimes the output from previous runs. I suspect that when
cInverter::query() does exceed the 2s timeout, it quits, and leaves the
response of its commands on the wire. For fd = /dev/hidraw0
tcflush(fd, TCOFLUSH);
has no effect.
As a matter of fact, on my system I have increased the timeout in
cInverter::query() to 15s and now it does always work correctly.
Also the program, when called with -1, could exit, while the thread
is receiving data, this leaves the next invocation with some
ready-data, which it is not expecting.
On Raspberry Pi 3B executing the original code, with Influx disabled, takes: real 0m57,753s user 0m54,679s sys 0m2,214s With the modification the time is: real 0m10,733s user 0m8,338s sys 0m0,759s For the record, I run the code outside of containers.
Here more jq-calls are replaced with sed. The time on Raspberry Pi is now real 0m4,965s user 0m2,595s sys 0m0,726s I do not use Influx. This allows updating the data in OpenHab (I read the MQTT stream from OpenHAB) every five seconds, once all jq calls are removed.
|
Some of the changes are also on https://github.com/manio/skymax-demo/ . |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This changeset accumulates:
and represents what I use on my system.