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

[smaenergymeter] Binding should support more that 1 device #3429

Closed
michaeljoos72 opened this issue Mar 30, 2018 · 6 comments · Fixed by #11718
Closed

[smaenergymeter] Binding should support more that 1 device #3429

michaeljoos72 opened this issue Mar 30, 2018 · 6 comments · Fixed by #11718
Labels
enhancement An enhancement or new feature for an existing add-on

Comments

@michaeljoos72
Copy link

Right now only 1 Thing is auto discovered....even if there are more that one energy meters in the network.
The result is, that the channels (power in, power out, energy in, energy out) are switching between the different energy meter values (according refresh interval).

It's already possible to add all energy meters to the Things-File. But with the same result. All Things are jumping between the values.

It would be good if the binding can filter the data according serial numbers. Maybe an additional parameter "serialnumber" for each thing?

The binding itself is very helpful, but with more than one meter it's not really usable.

My system:
openHAB 2.3 latest snapshot
2x SMA Energy Meter
Raspi3
@wborn
Copy link
Member

wborn commented Apr 3, 2018

@monnimeter could you implement these changes?

@wborn wborn changed the title SMA Energy Meter Binding should support more that 1 device [SMA Energy Meter] Binding should support more that 1 device Dec 15, 2018
@wborn wborn changed the title [SMA Energy Meter] Binding should support more that 1 device [smaenergymeter] Binding should support more that 1 device Dec 20, 2018
@davidgraeff davidgraeff added the enhancement An enhancement or new feature for an existing add-on label Mar 18, 2019
@hmerk
Copy link
Contributor

hmerk commented Jul 25, 2021

closed due to inactivity

@hmerk hmerk closed this as completed Jul 25, 2021
@FeKn
Copy link

FeKn commented Nov 7, 2021

I've the same problem:
The simultaneous operation of e.g. a SMA Energy Meter and a SMA Sunny Home Manager 2.0 in the same network is not possible, because the binding does not seem to evaluate the serial numbers in the Multicasts and therefore permanently updates the (wrong) values for all Things.

Is there a way to fix this?

@michaeljoos72
Copy link
Author

michaeljoos72 commented Nov 7, 2021 •
edited

No, not with this binding. Instead I'm using this:

https://github.com/datenschuft/SMA-EM

Does support serial numbers and different options to send the data. I'm using MQTT to send the data to openHAB.
(for MQTT you need to install: sudo pip3 install paho-mqtt)

@openhab-bot
Copy link
Collaborator

This issue has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/sma-energy-meter-binding-yields-unplausible-values/128180/5

splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Dec 5, 2021
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
cef010c 
Added support for multiple meters in single multicast group openhab#3429.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
@splatch splatch mentioned this issue Dec 5, 2021
Merged
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Dec 5, 2021
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
9cf6061 
Added support for multiple meters in single multicast group openhab#3429.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Dec 5, 2021
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
4bf913c 
Added support for multiple meters in single multicast group openhab#3429.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Dec 6, 2021
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
ea41b31 
Added support for multiple meters in single multicast group openhab#3429.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Dec 13, 2021
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
7ce1e6b 
Added support for multiple meters in single multicast group openhab#3429.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Dec 13, 2021
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
9d0c4d5 
Added support for multiple meters in single multicast group openhab#3429.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Dec 13, 2021
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
0ab493c 
Added support for multiple meters in single multicast group openhab#3429.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Apr 23, 2022
@splatch
[smaenergymeter] Ignore frames which are using protocol identifier di…
6cd2075 
…fferent than e-meter.

Fix handling of broadcast frames for SMA meter openhab#11497.

Added support for multiple meters in single multicast group openhab#3429.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Nov 6, 2022
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
d882891 
Added support for multiple meters in single multicast group openhab#3429.
Ignore frames which are using protocol identifier different than e-meter.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Nov 6, 2022
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
492d090 
Added support for multiple meters in single multicast group openhab#3429.
Ignore frames which are using protocol identifier different than e-meter.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Nov 6, 2022
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
7020fca 
Added support for multiple meters in single multicast group openhab#3429.
Ignore frames which are using protocol identifier different than e-meter.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Nov 6, 2022
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
9fbd465 
Added support for multiple meters in single multicast group openhab#3429.
Ignore frames which are using protocol identifier different than e-meter.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Nov 6, 2022
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
e9a2258 
Added support for multiple meters in single multicast group openhab#3429.
Ignore frames which are using protocol identifier different than e-meter.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Nov 6, 2022
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
87e13a6 
Added support for multiple meters in single multicast group openhab#3429.
Ignore frames which are using protocol identifier different than e-meter.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Nov 6, 2022
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
251bd9a 
Added support for multiple meters in single multicast group openhab#3429.
Ignore frames which are using protocol identifier different than e-meter.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Nov 27, 2022
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
7ada294 
Added support for multiple meters in single multicast group openhab#3429.
Ignore frames which are using protocol identifier different than e-meter.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
@jlaur jlaur linked a pull request Dec 10, 2022 that will close this issue
[smaenergymeter] Fix handling of broadcast frames #11718
Merged
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Jan 8, 2023
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
8712d1a 
Added support for multiple meters in single multicast group openhab#3429.
Ignore frames which are using protocol identifier different than e-meter.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Jan 8, 2023
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
e800a49 
Added support for multiple meters in single multicast group openhab#3429.
Ignore frames which are using protocol identifier different than e-meter.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Jan 8, 2023
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
61e9c5a 
Added support for multiple meters in single multicast group openhab#3429.
Ignore frames which are using protocol identifier different than e-meter.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Sep 21, 2023
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
4d975ca 
Added support for multiple meters in single multicast group openhab#3429.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Sep 21, 2023
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
cbf7eba 
Added support for multiple meters in single multicast group openhab#3429.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Sep 21, 2023
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
7c2be75 
Added support for multiple meters in single multicast group openhab#3429.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Sep 21, 2023
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
75982ab 
Added support for multiple meters in single multicast group openhab#3429.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
@lsiepel lsiepel reopened this Sep 22, 2023
@lsiepel
Copy link
Contributor

lsiepel commented Sep 22, 2023

re-opened: Issue has an active PR attached.

splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Jan 2, 2024
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
1e44c55 
Added support for multiple meters in single multicast group openhab#3429.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
splatch added a commit to ConnectorIO/openhab-addons that referenced this issue Jan 2, 2024
@splatch
Fix handling of broadcast frames for SMA meter openhab#11497.
1f92010 
Added support for multiple meters in single multicast group openhab#3429.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
@lsiepel lsiepel mentioned this issue Feb 4, 2024
Open
lolodomo pushed a commit that referenced this issue May 10, 2024
@splatch @lsiepel
[smaenergymeter] Fix handling of broadcast frames (#11718)
3c0eb94 
* Fix handling of broadcast frames for SMA meter #11497.

Added support for multiple meters in single multicast group #3429.

Signed-off-by: Łukasz Dywicki <luke@code-house.org>
Co-authored-by: Leo Siepel <leosiepel@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[smaenergymeter] Fix handling of broadcast frames ConnectorIO/openhab-addons
7 participants
@davidgraeff @hmerk @openhab-bot @wborn @lsiepel @FeKn @michaeljoos72