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

Sofar: Modify template for LSE-3 logger stick #11222

Merged
merged 1 commit into from
Dec 23, 2023
Merged

Conversation

premultiply
Copy link
Member

@premultiply premultiply commented Dec 23, 2023

Fixes #10723

/cc @cschlipf

@premultiply premultiply merged commit d974d37 into master Dec 23, 2023
6 checks passed
@premultiply premultiply deleted the device/sofar-lse3 branch December 23, 2023 21:50
id: 1
- name: delay
default: 1s
default: 0s
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@premultiply ein Default von 0 ist ein nop- das kann auch raus.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich finde es besser es drin zu lassen passend zur aktualisierten Dokumentation. Das ist nun ja ein Template auch für direkten RS485 Anschluss, wo man weiterhin das Delay braucht.

Copy link
Contributor

@cschlipf cschlipf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sieht gut aus.

id: 1
- name: delay
default: 1s
default: 0s
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich finde es besser es drin zu lassen passend zur aktualisierten Dokumentation. Das ist nun ja ein Template auch für direkten RS485 Anschluss, wo man weiterhin das Delay braucht.

@cschlipf
Copy link
Contributor

Hi @premultiply - sehe jetzt nach dem Release die Doku und hätte da einen Vorschlag zur Änderung:
image

Der RS485 über TCP macht hier nicht viel Sinn mit dem Port des LSE-3 Sticks, würde ich daher weglassen. Oben würde ich noch einfügen, dass man den LSE3 über Modbus TCP konfigurieren soll und statt 'hostname' im Kommentar würde ich vielleicht "Hostname oder IP Adresse des LSE-3 Logger Sticks" schreiben.

Ich glaube jemand der hier völlig unbedarft ist, wird das sonst nicht hinbekommen.

@premultiply
Copy link
Member Author

Die Doku wird automatisch aus den Templates erzeugt.
Da gibt es nichts manuell zu editieren.
Es werden alle möglichen Transportvarianten angezeigt.
Das Template muss auch abwärtskompatibel sein.

@egge96
Copy link

egge96 commented Feb 25, 2024

Ich nutze RS485 und bin zufrieden damit, leider funktioniert das nach dem Update auf 0.124.4 nicht mehr. Aus der Doku heraus habe ich nicht verstanden, was ich als Nutzer von RS485 (USB) jetzt anpassen muss. Muss ich wieder einen Delay-Parameter einfügen?

@andig
Copy link
Member

andig commented Feb 25, 2024

@egge96 das hat hier nichts zu suchen. Siehe #12335 und bitte neue Diskussion dazu.

@evcc-io evcc-io locked as off-topic and limited conversation to collaborators Feb 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extend Sofar template
4 participants