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

"main.js" update. Bessere Performance, Fehlerbehebung / Better performance, bugfixes #553

Open
PiXasCoding opened this issue May 4, 2024 · 2 comments

Comments

@PiXasCoding
Copy link

PiXasCoding commented May 4, 2024

(Scroll down for english text)

Wie unter folgendem Link diskutiert, habe ich Änderungen an der "main.js" programmiert.
#382

  • Die CPU-Last ist nun deutlich niedriger
  • Datenpunkte werden nun auch aktualisiert wenn man unter "Puffer in Sekunden, um Nachrichten zu ignorieren" einen Wert in Sekunden einträgt

Meine Veränderungen habe ich wie folgt markiert mit und sind zur besseren Übersicht erstmal nicht eingerückt:
// ******** Code change by PixasCoding START ********
// ******** Code change by PixasCoding END **********

@DutchmanNL
Bitte anschauen und wenn möglich in den Adapter einbauen

Die geänderte "main.js" kann hier eingesehen und heruntergeladen werden:
https://github.com/PiXasCoding/ioBroker.vedirect_update/tree/main

Die Datei gehört in folgendes Verzeichnis:
/opt/iobroker/node_modules/iobroker.vedirect/

-------------- English text --------------
As discussed under the following link, I have programmed changes to the "main.js".
#382

  • The CPU load is now significantly lower
  • Data points are now also updated if you enter a value in seconds under "Buffer in seconds to ignore messages".

I have marked my changes as follows and have not indented them for the time being for a better overview:
// ******** Code change by PixasCoding START ********
// ******** Code change by PixasCoding END **********

@DutchmanNL
Please take a look and install it in the adapter if possible

The modified "main.js" can be viewed and downloaded here:
https://github.com/PiXasCoding/ioBroker.vedirect_update/tree/main

The file belongs in the following directory:
/opt/iobroker/node_modules/iobroker.vedirect/

@saeft2003
Copy link

@PiXasCoding vielen dank für die Datei ich hab das ganze jetzt getestet und bin sehr zufrieden.

Ich habe auf einem extra host mit proxmox zwei VMs mit iobroker laufen in diesem laufen zusammen 7 vedirect instanzen. Per blockly schicke ich die Daten an mein haupt iobroker. Somit hatte ich die ganze Zeit die hohe CPU Auslastung "ausgelagert".

Die CPU Auslastung vom host ist jetzt mit 10 sek Puffer von 26% auf 13% gesunken. Der RAM ist gesunken von 7GB auf 4GB.

Unbenannt

@DutchmanNL ich würde das sehr beführworten, dass das eingebaut wird, damit die anderen auch einen nutzen davon haben.

@PiXasCoding
Copy link
Author

Poh!
CPU Auslastung um 50% gesunken, RAM um 43% niedriger! Das ist ja schon richtig viel was das bei deiner Konfiguration ausmacht.
Ich bin mir ziemlich sicher, dass man sogar noch etwas mehr rauskitzeln könnte. Z.B. während der Wartezeit die USB-Daten gar nicht erst empfängt. Erstmal abwarten was DutchmanNL dazu schreibt.

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