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

Add Teleinfo Binding #680

Closed
nokyyz opened this issue Feb 25, 2016 · 24 comments
Closed

Add Teleinfo Binding #680

nokyyz opened this issue Feb 25, 2016 · 24 comments
Assignees
Labels
new binding If someone has started to work on a binding. For a new binding PR.

Comments

@nokyyz
Copy link

nokyyz commented Feb 25, 2016

Hi,
I'm working on new binding OH2 which enables to manage data from French electricity meter (http://www.magdiblog.fr/wp-content/uploads/2014/09/ERDF-NOI-CPT_02E.pdf) : Teleinfo binding.

This binding will have the following features:

  • retrieves ERDF frames from a serial port
  • retrieves ERDF frames from a remote serial port (server)
  • management of all data types : ADCO, OPTARIF, HCHC, HCHP, ISOUSC, IINST, IMAX, PAPP, PTEC, ADPS, MOTDETAT, etc.
  • calculates the cost of power consumption

Any help will be appreciate !

@kaikreuzer kaikreuzer added the new binding If someone has started to work on a binding. For a new binding PR. label Feb 25, 2016
@srsr
Copy link

srsr commented Jan 18, 2017

Hello,
I am very interested in this module, have you progressed?

@mgallissot
Copy link

HI,
Interested by this new binding, how can I help?

@vdanjean
Copy link

Hi,
Are there any news or a git with preliminary code somewhere (so that we can participate)?

@kaikreuzer
Copy link
Member

@vdanjean The code seems to be available at https://github.com/nokyyz/teleinfo-reader, but there wasn't any activity since 2016...

Not sure whether anyone would be interested to take this over and build on it? If not, I'd suggest to close this issue here.

@nokyyz
Copy link
Author

nokyyz commented Jun 12, 2018

Hi all, I am sorry but I have no enough free time to work on this plugin. The source code on my repository (https://github.com/nokyyz/teleinfo-reader) enables to read teleinfo frames and broadcast them on network.

@wborn wborn changed the title Teleinfo binding Add Teleinfo Binding Dec 19, 2018
@nokyyz
Copy link
Author

nokyyz commented Jul 5, 2019

Hi all, I have restarted my work on Teleinfo binding ! I think that in 2/3 weeks, I will have a first alpha release compliant with OH 2.5.
For the moment, I read all Teleinfo data from my Electrical meter with HC/HP option.

@srsr @mgallisot @vdanjean Can volunteers help me for tests on other types of meters (Base, Tempo, EJP options) ?

@nokyyz
Copy link
Author

nokyyz commented Jul 5, 2019

@davidgraeff @wborn is it possible to reopen this issue please ?

@wborn
Copy link
Member

wborn commented Jul 5, 2019

That's good news! I've reopened the issue. :-)

@wborn wborn reopened this Jul 5, 2019
@nokyyz
Copy link
Author

nokyyz commented Jul 11, 2019

I just created a thread to discuss the binding and its features:
https://community.openhab.org/t/new-teleinfo-binding-tester-and-feedback-welcome/77857

@srsr
Copy link

srsr commented Sep 3, 2019

Hello nokyyz,
Sorry I own a HC/HP option too.

@nokyyz
Copy link
Author

nokyyz commented Sep 3, 2019

Hi @srsr, have you planned to test this binding ? :-) The Teleinfo binding is working on my platform (hardware: GCE teleinfo module, Synology NAS 415+ & software : DSM 6.x, OpenHAB 2.5) but I will appreciate some feedbacks from differents platforms...

@nokyyz
Copy link
Author

nokyyz commented Nov 4, 2019

Hi all,
Here is the new release (Beta 20191104-2035) : https://github.com/nokyyz/openhab2-addons/releases/tag/beta-20191104-2035

Any feedback will be welcome !

@vdanjean
Copy link

vdanjean commented Nov 8, 2019

Hi,
Thank you very much for this development. I will try it soon. Before looking in the sources, I've a question: my serial controler is plugged into a physical machine, however my OpenHAB run into a virtual machine. So, it there a simple way to configure this plugins to communicate through TCP/IP (for example with socat running on the physical machine to relay the data)?
Regards

@nokyyz
Copy link
Author

nokyyz commented Nov 8, 2019

Hi @vdanjean,
The latest release (Beta 20191104-2035) of Teleinfo binding must support the RFC2217 protocol. I can't test this feature but I have copied the source code of EnOcean binding for this RFC2217 protocol, so it should be OK. For more details, see the EnOcean binding documentation at https://www.openhab.org/addons/bindings/enocean/#concepts-configuration.
Note: ser2net command seems to be used instead of socat command

@vdanjean
Copy link

Hi,
I should have a bit of time to do tests. Do you have some documentation about how to compile the teleinfo binding? I'm using openhab packages from https://dl.bintray.com/openhab/apt-repo2 (currently, version 2.5.3-1)
Regards,
Vincent

@nokyyz
Copy link
Author

nokyyz commented Apr 19, 2020

Hi @vdanjean,
I have compile the latest version of Teleinfo binding. You can download it here : https://github.com/nokyyz/openhab2-addons/releases/tag/beta-20200204-1329

To compile OpenHAB sources and Teleinfo binding, you must have Apache Maven. From memory, the 'mvn install' command on root sources folder must allow to generate a Teleinfo JAR file.

@fwolter
Copy link
Member

fwolter commented Jun 3, 2020

@nokyyz @olivierkeke Can you give a short overview, why there are to open PRs? Which one should be reviewed/merged? What is the difference between the two?

@olivierkeke
Copy link
Contributor

@fwolter The two PR are quite the same.

In #7516 @nokyyz commits and my commits are mixed with upstream commits since we made some merge with upstream instead of rebasing on upstream. This prevents us to sign off our commits with a simple command.

In #7744 commit history is clean: our commits are at the last commits in the history. Since I have no news of @nokyyz for several weeks and no right on the branch of the initial PR, I made a separate PR. I also rebased our work on the last 2.5.6-SNAPSHOT version. AFAIK #7744 should be reviewed.

@fwolter
Copy link
Member

fwolter commented Jun 4, 2020

@nokyyz If you agree with @olivierkeke, I would review #7744.

@nokyyz
Copy link
Author

nokyyz commented Jun 4, 2020

Hi @fwolter, @olivierkeke,
After analysing @olivierkeke commit clean history in #7744, I see that all of my commits have been sign-off by Olivier. Is it possible to fix this ?

@olivierkeke
Copy link
Contributor

Hi @nokyyz,
Glad to read you again!
If you launch git rebase HEAD~78 --signoff on the PR branch you will add your signature on all our commits (without removing mine AFAIK).
After that you can push your change with git push --force-with-lease origin rebase-2.5.x.
I have added you as collaborator on my fork repo.
@fwolter Will this be OK?

@fwolter
Copy link
Member

fwolter commented Jun 5, 2020

It should work this way, although I didn't count the HEAD~78.

@olivierkeke
Copy link
Contributor

@nokyyz Since teleinfo binding is now an official addon, can you close this issue?

@nokyyz
Copy link
Author

nokyyz commented Sep 22, 2020

Of course. Done !

@nokyyz nokyyz closed this as completed Sep 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new binding If someone has started to work on a binding. For a new binding PR.
Projects
None yet
Development

No branches or pull requests

9 participants