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.

Sign up for GitHub

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

Jump to bottom

performance optimization #52

Open
wants to merge 20 commits into
base: master
Choose a base branch
from
Open

performance optimization #52

wants to merge 20 commits into from

Commits on Sep 29, 2021

  1. move constant declarations outside the function calls, no need to re-run 

    them for each call.
    @kchiem
    kchiem committed Sep 29, 2021
    Configuration menu
    Copy the full SHA
    9ffed7b View commit details
    Browse the repository at this point in the history

Commits on Oct 2, 2021

  1. - loop the subscriber process to handle mqtt server restarts 

    - reduce unnecessary wrapper processes
    @kchiem
    kchiem committed Oct 2, 2021
    Configuration menu
    Copy the full SHA
    73aa986 View commit details
    Browse the repository at this point in the history

  2. change temperature icon

    @kchiem
    kchiem committed Oct 2, 2021
    Configuration menu
    Copy the full SHA
    7dd9283 View commit details
    Browse the repository at this point in the history

  3. update mqtt immediately after running a command

    @kchiem
    kchiem committed Oct 2, 2021
    Configuration menu
    Copy the full SHA
    be2d786 View commit details
    Browse the repository at this point in the history

  4. - @nrm21 inspired changes--allows sending commands longer than 5 byte… 

    …s and

removes the need to provide qpiri/qpiws/qmod/qpigs reply sizes in the
config.
- also fixed some compiler warnings.
    @kchiem
    kchiem committed Oct 2, 2021
    Configuration menu
    Copy the full SHA
    dbea469 View commit details
    Browse the repository at this point in the history

  5. - @nrm21 inspired changes--allows sending commands longer than 5 byte… 

    …s and removes the need to provide qpiri/qpiws/qmod/qpigs reply sizes in the config.

- also fixed some compiler warnings.
    @kchiem
    kchiem committed Oct 2, 2021
    Configuration menu
    Copy the full SHA
    5cb57c5 View commit details
    Browse the repository at this point in the history

  6. Merge branch 'master' of github.com:kchiem/docker-voltronic-homeassis… 

    …tant
    @kchiem
    kchiem committed Oct 2, 2021
    Configuration menu
    Copy the full SHA
    8d548e4 View commit details
    Browse the repository at this point in the history

  7. fix typo

    @kchiem
    kchiem committed Oct 2, 2021
    Configuration menu
    Copy the full SHA
    8c00a51 View commit details
    Browse the repository at this point in the history

  8. switch to while loops so that the output can be read in docker logs.

    @kchiem
    kchiem committed Oct 2, 2021
    Configuration menu
    Copy the full SHA
    8b79ebb View commit details
    Browse the repository at this point in the history

  9. - @nrm21 inspired changes--allows sending commands longer than 5 byte… 

    …s and

removes the need to provide qpiri/qpiws/qmod/qpigs reply sizes in the
config.
- also fixed some compiler warnings.
    @kchiem
    kchiem committed Oct 2, 2021
    Configuration menu
    Copy the full SHA
    6d7e649 View commit details
    Browse the repository at this point in the history

  10. Merge branch 'master' of github.com:kchiem/docker-voltronic-homeassis… 

    …tant
    @kchiem
    kchiem committed Oct 2, 2021
    Configuration menu
    Copy the full SHA
    25c806c View commit details
    Browse the repository at this point in the history

Commits on Oct 4, 2021

  1. reduce the number of jq invocations drastically 

    convert json output from the poller directly into a bash hash, so that
it can be iterated upon.
    @kchiem
    kchiem committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    6ffef39 View commit details
    Browse the repository at this point in the history

  2. remove the push after running a command, didn't like the extra delay. 

    added publishing the poller's reply into mqtt.
    @kchiem
    kchiem committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    043d07e View commit details
    Browse the repository at this point in the history

  3. don't need the fixed response sizes anymore.

    @kchiem
    kchiem committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    25d9aaf View commit details
    Browse the repository at this point in the history

Commits on Oct 5, 2021

  1. set retain flag on publishes, don't need to loop the init script 

    anymore with default HA install of Mosquitto broker using persistence.
    @kchiem
    kchiem committed Oct 5, 2021
    Configuration menu
    Copy the full SHA
    60186ea View commit details
    Browse the repository at this point in the history

  2. add timestamps to stdout output (for docker logs) 

    retry up to 3 times total if the reply is anything under than "ACK"
    @kchiem
    kchiem committed Oct 5, 2021
    Configuration menu
    Copy the full SHA
    4eef447 View commit details
    Browse the repository at this point in the history

  3. updated for watch removal.

    @kchiem
    kchiem committed Oct 5, 2021
    Configuration menu
    Copy the full SHA
    711370b View commit details
    Browse the repository at this point in the history

Commits on Oct 7, 2021

  1. fixed a bug with response handling giving corrupt values. 

    don't accept "NAK" results for QMOD, QPIGS, QPIRI, QPIWS queries on
polling.

clean up debug output a little.
    @kchiem
    kchiem committed Oct 7, 2021
    Configuration menu
    Copy the full SHA
    c8b560d View commit details
    Browse the repository at this point in the history

  2. use a later Debian image for a newer mosquitto-clients package.

    @kchiem
    kchiem committed Oct 7, 2021
    Configuration menu
    Copy the full SHA
    8296b07 View commit details
    Browse the repository at this point in the history

  3. oops, can't break out of if-blocks.

    @kchiem
    kchiem committed Oct 7, 2021
    Configuration menu
    Copy the full SHA
    9a06aa9 View commit details
    Browse the repository at this point in the history