Skip to content
This repository has been archived by the owner on Jun 25, 2019. It is now read-only.

MQTT Monitor and new SDM120 MeterType (Reduced Data from SDM) #22

Closed
wants to merge 4 commits into from

Conversation

mreschka
Copy link

Hi,

I have created a monitor based on your sample monitor which pushes all data from the meters to a MQTT Host using the eclipse paho go lib for MQTT. Since I have many SDM120 Meters I introduced a new METER-Sub-Type SDM120 as well to reduce the useless data in MQTT. Feel free to simply ignore that part of the pull request. Sadly it is not possible to split the pull request since I committed both before I realized that.

I am new to golang, so please especially check what I did to the vendor directory. I did not manage to make the project without those additions (instead of lcal library installations) but maybe you would prefer another release of the three new packages - I simply added the master branches.

Thank's a lot for your work!!!!

Cheers,

Markus

Added MQTT Monitor using paho library
Added a new Type: SDM120, which reads far less data (only Phase A)
The maximum update rate can nw be defined. Default=0=unlimited. That means after a push we will ignore more data from same device and channel for this time. The real update rat will be longer than that time since we will wait for new data to come from the firehose to push after the time has expired.
@andig andig mentioned this pull request Feb 11, 2018
@andig
Copy link
Collaborator

andig commented Feb 11, 2018

@mreschka see #25 for a clean PR with on MQTT.

You should never work in the master branch for a forked repo. To correct use git reset to go back, then create a new branch per feature and commit only changes for a single feature to that branch.

@mreschka mreschka closed this Feb 11, 2018
@mreschka
Copy link
Author

thnanks andig for sorting this out :-)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants