Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
E3DC nun komplett auf openwb V2.0 Struktur umgestellt #2449
E3DC nun komplett auf openwb V2.0 Struktur umgestellt #2449
Changes from 7 commits
7858789
acf50f4
7d2fdbb
b7e27f1
4fec76f
ddbe6aa
502a9d7
6809da8
6a38f02
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
This file was deleted.
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hier sollte nur eine Adresse sein. Die ich auch wieder nur
address
nennen würde, denn es könnte auch ein hostname sein.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ich frage mich bei dieser Funktion immernoch, ob das cachen des nötigen Registers tatsächlich ein Vorteil ist. So könnte die Funktion ohne aussehen:
Das ist schon deutlich weniger (der Kommentar der erklärt was der Kram soll mit den
meters
sollte da natürlich wieder rein, müsste dafür aber stimmen.). Dazu kommt noch, dass die Funktion in V2 so nicht funktioniert, weil es keine Ramdisk mehr gibt. Das kann man natürlich lösen indem das Register als Eigenschaft auf dem Counter-Objekt gespeichert wird, was allerdings in V1 nicht geht. Also muss da wieder eine Unterscheidung rein, wodurch es noch mehr Code wird.Die Kosten für die kürzere Variante ist etwas Performance. Das ganze kostet ein paar zusätzliche Millisekunden (wenn man davon ausgeht, dass dass da ein Modbus-RS485-Kopple mit 9600 Baud hinter hängt komme ich auf eine ganze grobe Größenordnung von 20 Millisekunden). Mir erscheint das die Vereinfachung wert zu sein.
Natürlich funktioniert meine Vereinfachung nur, wenn der Server es erlaubt 28 Register auf einen Schlag auszulesen. Das müsstest du ggf. nachlesen oder ausprobieren.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Phasenleistung"? Auf einem Register? Sollte wohl eher 40130-40132. 1 register je Phase.
Stimmt das wirklich? Gibt der e3dc da nur einen Ganzzahligen Wert raus? Das wäre ziemlich ungenau.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
max 6 Leistungsmesser? Die Schleife liest aber 7 aus.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Diese Zeile vom Kommentar scheint mir keine Hilfe zu sein.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gleicher Kommentar zu
ip_address
wie inE3dcBat