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

AirCondition DP Swing doppelt, überschreibt sich selbst im Alias #356

Open
tt-tom17 opened this issue Oct 24, 2024 · 4 comments
Open

AirCondition DP Swing doppelt, überschreibt sich selbst im Alias #356

tt-tom17 opened this issue Oct 24, 2024 · 4 comments

Comments

@tt-tom17
Copy link

Describe the bug
https://github.com/ioBroker/ioBroker.type-detector/blob/ad56c80de738870cfd94cae4950c4b69859e0327/src/TypePatterns.ts#L1439

es gibt zwei Rollen level.mode.swing und switch.mode.swing die den selben Datenpunkt anlegen und damit sich selbst überschreiben. Das Ergebnis sieht dann so aus.

{
"_id": "alias.0.NSPanel_EMU.TestKlimaVirtual.SWING",
"native": {},
"type": "state",
"common": {
"alias": {
"id": "0_userdata.0.Test.Klimaanlage.Swing2"
},
"name": "SWING",
"role": "switch.mode.swing",
"write": true,
"states": {
"0": "AUTO",
"1": "HORIZONTAL",
"2": "STATIONARY",
"3": "VERTICAL"
},
"type": "boolean"
},
"acl": {
"object": 1636,
"state": 1636,
"owner": "system.user.admin",
"ownerGroup": "system.group.administrator"
},
"from": "system.adapter.admin.0",
"user": "system.user.admin",
"ts": 1729771316211
}

im Device-Manager weden auch beide Rollen gleichzeitig der gewählte DP gesetzt. Die Anmerkung/Hinweis auf die Rolle unter dem DP-Feld stimmen auch nicht.
Bildschirmfoto 2024-10-24 um 14 05 14

To Reproduce
Steps to reproduce the behavior:
Geräte-Manager eine Klimaanlage erstellen, besondere Beachtung beim Punkt "Swing"

Versions:

  • Adapter version: 1.1.5 Geräte-Manager
  • JS-Controller version: 7.0.1
  • Node version: 20.18.0
  • Operating system: Debian 12.0
@Garfonso
Copy link
Contributor

Thanks for creating this issue. It is more an issue with devices adapter, though. Since the pattern is allowed and AirCondition is not the only devices that applies it. I'll try to look into it.

@tt-tom17
Copy link
Author

Wäre Swinglevel und Swingstate nicht sinnvoller als nur Swing. Der Device Manager zieht sich die Daten/Config aus dem typedetector.

@Garfonso
Copy link
Contributor

Garfonso commented Nov 25, 2024

Zumindest bei den states, die das Objekt oben hat, wäre "Swinglevel" aber nicht passend.

Der Device Manager zieht sich die Daten/Config aus dem typedetector.

Ja, aber wie gesagt, es ist in den Pattern erlaubt, den gleichen state mehrfach zu haben mit verschiedenen typen. Insofern ist das kein Fehler im type-detector, das Pattern ist so richtig. Wenn du einen zustätzlichen State im Pattern vorschlagen willst, ist das natürlich was anderes. Da bräuchtest du dann eine passende Rolle usw. Mach das gerne.
Allerdings wäre Swinglevel für den State, den du oben hast, m.E. nicht sehr passend.

Und der Fall "an,aus,auto" oder sowas, passt halt für den State. Meiner Meinung nach auch die Optionen, die du oben hast.

@tt-tom17
Copy link
Author

okay, wenn das Pattern so erlaubt ist, muss der Fehler im Device-Manager gesucht werden. Ich müsste dann eine Möglichkeit haben die richtige Rolle auszuwählen und ein überschreiben verhindert werden. Es müsste für jede Rolle auch ein eigener Aliasdatenpunkt erstellt werden. Dies wird auch nicht gemacht, was ein überschreiben auslöst.

@Apollon77 Apollon77 transferred this issue from ioBroker/ioBroker.type-detector Nov 26, 2024
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

No branches or pull requests

2 participants