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

Alphatec: Add configurable timeout #8523

Merged
merged 6 commits into from
Jun 18, 2023
Merged

Conversation

premultiply
Copy link
Member

May fix the EOF error discussed in #6124 due to slow device response

@deadrabbit87 @Matwas82 Bitte mal im Nightly mit höheren Werten (>= 10s) testen ob sich eine Verbesserung einstellt und die EOF-Fehler dann verschwinden.

@premultiply premultiply added the devices Specific device support label Jun 18, 2023
@premultiply premultiply self-assigned this Jun 18, 2023
@premultiply premultiply changed the title Alpatec: Add configurable timeout Alphatec: Add configurable timeout Jun 18, 2023
@premultiply premultiply merged commit e50ac98 into master Jun 18, 2023
@premultiply premultiply deleted the device/alphatec-timeout branch June 18, 2023 12:04
@deadrabbit87
Copy link
Contributor

Mach ich morgen gleich, danke dir!

@deadrabbit87
Copy link
Contributor

deadrabbit87 commented Jun 19, 2023

Ich hab jetzt <= 20 s getestet. Die Config:

chargers:
  - name: AW1eM
    type: template
    template: alphatec
    modbus: rs485serial
    id: 1
    device: /dev/ttyACM0
    baudrate: 9600
    comset: "8N1"
    timeout: 20s

Mir ist aufgefallen, dass in der log trace was von 5s steht? Gehört das so?

log.txt

@andig
Copy link
Member

andig commented Jun 19, 2023

@deadrabbit87 hast du nur genau dieses eine Modbusgerät? Oder noch Zähler etc?

@deadrabbit87
Copy link
Contributor

Kostal Plenticore wird auch via TCP Modbus ausgelesen. Die WB aber via USB RS485 Umsetzer.

@andig
Copy link
Member

andig commented Jun 19, 2023

Dann ist TCP die Quelle für die 5s Meldung und hat hiermit nix zu tun...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devices Specific device support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants