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

first version of OMS based meter support #181

Merged
merged 9 commits into from Jul 12, 2015

Conversation

mbehr1
Copy link
Contributor

@mbehr1 mbehr1 commented Jul 11, 2015

As discussed in #176 see here the first version.
Thanks to @schnello for support with infos and debug host.

Currently only 4 Obis values are supported: 1.7.0, 2.7.0, 1.8.0 and 2.8.0.

For configuration use conf editor or

{
"protocol":  "oms",
"device": "/dev/ttyUSB0",
"key": "<32char AES key in hex without spaces>"
"channel": { "uuid": "...", "identifier": "1.7.0" }
}

.
Currently only serial/tty devices and encryption mode 5 is supported.

Please don't use interval but use aggregation to reduce data rate if needed/wanted.

It requires openssl and libmbus >=0.8.0 from e.g. https://github.com/rscada/libmbus.git
(installation is easy:

git clone https://github.com/rscada/libmbus.git
cd libmbus
./build.sh
make install

Please test and let me know any feedback.

@mbehr1
Copy link
Contributor Author

mbehr1 commented Jul 12, 2015

I fixed a hang on program termination (in non daemon mode after pressing ctrl-c) that was introduced with push data but found during tests here (thanks to @schnello).
Dumped version to 0.4.3. to reflect support of new meter.

@andig
Copy link
Contributor

andig commented Jul 12, 2015

Do we need libmbus support in install.sh?

@mbehr1
Copy link
Contributor Author

mbehr1 commented Jul 12, 2015

Would make sense (but otherwise build process will be ok but just with OMS support). Can you add it? Needed steps see above (git clone ..., ./build.sh make install)

@andig
Copy link
Contributor

andig commented Jul 12, 2015

Can you add it?

Glad to. I'll merge first and add install.sh then, ok?

@mbehr1
Copy link
Contributor Author

mbehr1 commented Jul 12, 2015

sure

@andig andig mentioned this pull request Jul 12, 2015
@andig
Copy link
Contributor

andig commented Jul 12, 2015

We're also missing a config example? I'll add one later on. Json editor should not be missed.

@mbehr1
Copy link
Contributor Author

mbehr1 commented Jul 12, 2015

I’ll add one. Had it in the first version but needed to remove as otherwise tests fail without libmbus…

Am 12.07.2015 um 14:09 schrieb andig notifications@github.com:

We're also missing a config example?


Reply to this email directly or view it on GitHub #181 (comment).

Gruß

Matthias

@andig
Copy link
Contributor

andig commented Jul 12, 2015

+1 for merging this pr. libmbus is merged.

@UdoSchake
Copy link

What abt. openssl? Don't we need it too?

Gruß
Udo

@mbehr1
Copy link
Contributor Author

mbehr1 commented Jul 12, 2015

openssl is already part for SML based readers.

@mbehr1
Copy link
Contributor Author

mbehr1 commented Jul 12, 2015

But there is one issue with the timestamps under investigation. I wait for merge until that's sorted out.

@UdoSchake
Copy link

openssl is already part for SML based readers.

Really? I dont find it in the dependencies of vzlogger, libsml or middleware.

Gruß
Udo

@mbehr1
Copy link
Contributor Author

mbehr1 commented Jul 12, 2015

sorry. is needed for mysmartgrid not for SML
excerpt from CMakeList.txt

 include(FindGnutls)
  include(FindOpenSSL) # needed by MySmartGrid API...

@mbehr1
Copy link
Contributor Author

mbehr1 commented Jul 12, 2015

Still ok for merge? I added conf file and fixed the timestamp issue reported from @schnello.

@mbehr1
Copy link
Contributor Author

mbehr1 commented Jul 12, 2015

Still one issue with timestamps (devices seems to send packets with duplicate ones). I'll add a workaround.

@mbehr1
Copy link
Contributor Author

mbehr1 commented Jul 12, 2015

I'll merge now so that we might get more people to test it. Different meters than the one from @schnello will most likely show new effects. Just let me know in case of any problems.

mbehr1 added a commit that referenced this pull request Jul 12, 2015
first version of OMS based meter support
@mbehr1 mbehr1 merged commit 343b19a into volkszaehler:master Jul 12, 2015
@mbehr1 mbehr1 deleted the MeterOMSv1 branch July 12, 2015 21:09
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

Successfully merging this pull request may close these issues.

None yet

3 participants