search

iobroker-community-adapters / ioBroker.openknx

free KNX Adapter for ioBroker
GNU General Public License v3.0
31 stars 4 forks source link

Provide option to set sending physical address (PA) in OpenKNX adapter configuration (again) - please #257

Open felsnerino opened 2 years ago

felsnerino commented 2 years ago

Is your feature request related to a problem? Please describe. Problem is, that I am using Open KNX Adapter with Multicast on the IP Backbone, which works overall great. The Problem is, that the filtering in IP Router is not working properly, as the sending PA of the IOBroker Adapter is 0/0/0. I have configured a dummy device in KNX, for example 1/0/10. I would like that the adapter is then sending from this PA.

Describe the solution you'd like Provide the option to configure, from which PA the adapter is sending ETS Packets. E.g. "1/0/10", etc.

Describe alternatives you've considered I could switch back to connect through the Router directly, but then I have to route ALL relevant traffic to this one KNX line, which is not useful. That is why I want to have an IP Backbone where I don't care about the sum of traffic from my 3 KNX Lines (outdoor, cellar and electrical distribution and the floors).

Additional context I think it was previously possible, or at least in the other KNX Adapter... but it would be really, really awesome to include it also in here, as I don't want to switch to the other one...

felsnerino commented 1 year ago

It would be really great to be able to set the PA. Especially if you have an actual "Bereichskoppler" in use, for example to connect my 3 IP Lines 1/1/x + 1/2/x+ 1/3/x it would be great if one could actually differentiate if the packets where send from the IObroker or my EIBPORT Server who is also the "Bereichskoppler" - 1.0.0... I wouldn't mind if it is not turn on by default, but it would be great if one could activate it if needed. Not sure why it was removed, as it doesn't really harm... Again, I very much appreciate your work and the adapter is really great! Thanks for all the work!!

stale[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs within the next 7 days. Please check if the issue is still relevant in the most current version of the adapter and tell us. Also check that all relevant details, logs and reproduction steps are included and update them if needed. Thank you for your contributions. Dieses Problem wurde automatisch als veraltet markiert, da es in letzter Zeit keine Aktivitäten gab. Es wird geschlossen, wenn nicht innerhalb der nächsten 7 Tage weitere Aktivitäten stattfinden. Bitte überprüft, ob das Problem auch in der aktuellsten Version des Adapters noch relevant ist, und teilt uns dies mit. Überprüft auch, ob alle relevanten Details, Logs und Reproduktionsschritte enthalten sind bzw. aktualisiert diese. Vielen Dank für Eure Unterstützung.

stale[bot] commented 1 year ago

This issue has been automatically closed because of inactivity. Please open a new issue if still relevant and make sure to include all relevant details, logs and reproduction steps. Thank you for your contributions. Dieses Problem wurde aufgrund von Inaktivität automatisch geschlossen. Bitte öffnet ein neues Issue, falls dies noch relevant ist und stellt sicher das alle relevanten Details, Logs und Reproduktionsschritte enthalten sind. Vielen Dank für Eure Unterstützung.

felsnerino commented 1 year ago

Is there really no chance to implement this? As it was already implement, I would assume, this is not complicated...

stale[bot] commented 1 year ago

This issue has been automatically closed because of inactivity. Please open a new issue if still relevant and make sure to include all relevant details, logs and reproduction steps. Thank you for your contributions. Dieses Problem wurde aufgrund von Inaktivität automatisch geschlossen. Bitte öffnet ein neues Issue, falls dies noch relevant ist und stellt sicher das alle relevanten Details, Logs und Reproduktionsschritte enthalten sind. Vielen Dank für Eure Unterstützung.

Ralf31337 commented 1 year ago

Parts of my smarthome stopped working because an update probably removed my workaround.

The workaround was: sed '/startKnxStack() {/,/}/{s/physAddr: "0.0.0"/physAddr: "1.0.253"/;}' main.js -i

If I can help with adding that feature (to UI, etc.), let me know. I would be available to do the coding and put a pull request.