search

home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
70.14k stars 29.18k forks source link

Modbus switch not working #55719

Closed prilly-dev closed 2 years ago

prilly-dev commented 2 years ago

The problem

Modbus switch is not working,

What is version of Home Assistant Core has the issue?

2021.9.2

What was the last working version of Home Assistant Core?

2021.3.x

What type of installation are you running?

Home Assistant Container

Integration causing the issue

Modbus

Link to integration documentation on our website

https://www.home-assistant.io/integrations/modbus

Example YAML snippet

# Systemair VTR250 - NEW CONFIG STYLE
modbus:
  - name: VTR250
    type: tcp
    host: 192.168.1.70
    port: 502
    #delay: 1
    #timeout: 1

############################################### SWITCHES ###############################################
    switches:

#-----------------------------------VTR250 MANUELL VIFTE STOPP TILLATT ON/OFF----------------

      - name: vtr250_manuell_stopp_tillatt
        address: 1352
        command_on: 1 
        command_off: 0 
        write_type: holding
        verify:

Anything in the logs that might be useful for us?

homeassistant.components.modbus.modbus] Pymodbus: VTR250: Exception Response(131, 3, IllegalAddress)

Additional information

modbus register doc: REG_FAN_MANUAL_STOP_ALLOWED I Holding Register (Function 03/06/16) 1352 0 1 Allow manual fan stop (also as selection for user modes and Week schedule). 0: Manual stop not allowed 1: Manual stop allowed

probot-home-assistant[bot] commented 2 years ago

modbus documentation modbus source (message by IssueLinks)

probot-home-assistant[bot] commented 2 years ago

Hey there @adamchengtkc, @janiversen, @vzahradnik, mind taking a look at this issue as it has been labeled with an integration (modbus) you are listed as a code owner for? Thanks! (message by CodeOwnersMention)

janiversen commented 2 years ago

Your device states it is an illegal address, are you sure it is a holding register?

Furthermore without a debug log I cannot tell you if the problem is when writing or when reading.

Some devices use e.g holding to write and coil to read (often at another address).

once you supply a debug log I can investigate further.

prilly-dev commented 2 years ago

Let me see if i can get a debug log,

Yes iam shure its a holding register, coils are NOT used in this device, adress is also correct and verified.

from docs of the modbus device:

.2 Transmission modes The registers are Input registers (read-only) and Holding registers (read-write) when using Modbus/RTU and Holding registers (read-write) when using Modbus/TCP. All registers are 16 bit Integers. Table 1 Signal types used Unsigned Integers I Signed Integers I* Table 2 Register types used Input Registers 16-bit quantity, Analog Input Register provided by an I/O system, read-only, Modbus Function 04 Holding registers 16-bit quantity, analog output HR alterable by an application program, read-write , Modbus function 03 / 06 / 16

janiversen commented 2 years ago

but your device still responds with “Illegal address”, try to add “slave: 0” or “slave: 1” it should be default, but I am not sure if its in the release

look in the modbus doc. it shows how to make a debug log.

prilly-dev commented 2 years ago

No need to use slave: 1 when modbus tcp is used or!?

2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_tillufts_temperatur with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_inntaks_temperatur with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_avtrekks_temperatur with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_overhetingstemperatur_tilluft with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_relativ_fukt_avtrekk with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_tilluft_sp with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_avtrekksluft_sp with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_avtrekksluft_min_sp with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_avtrekksluft_max_sp with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_eco_varme_offset with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_tilluftsvifte with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_avtrekksvifte with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_el_kolbe_padrag with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_el_kolbe_av_pa with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_varmegjenvinner with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_komfyravtrekk_trykkvakt_av_pa with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_modus_status_register with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_tilluft_rom_avtrekk_reg with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_manuell_modus_kommando_register with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_time_year with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_time_md with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_time_day with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_time_hour with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_time_min with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_time_sec with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_sch_active_temp_offset with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_sch_inactive_temp_offset with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_sch_prd1_day1_start_h with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_sch_prd1_day1_start_m with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_sch_prd1_day1_end_h with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_sch_prd1_day1_end_m with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_kalkulert_fukt_avtrekk with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_kalkulert_fukt_inntak with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_sommerdrift_vinterdrift with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_filterbytte_alm with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_filterbytte_md_intervall with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_tid_til_filter_bytte_s with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_nedtelling_modus_tid_s with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_ferie_modus_varighet with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_borte_modus_varighet with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_ildsted_modus_varighet with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_boost_modus_varighet with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_party_modus_varighet with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_fuktoverforing_sp with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_vifte_hast_komp_avlest with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_vifte_hast_komp_vinter with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_vifte_hast_komp_vinter_start_temp with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_vifte_hast_komp_vinter_max_temp with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_vifte_hast_komp_sjekkes with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_vifte_hast_komp_sommer with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_vifte_hast_komp_sommer_start_temp with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_vifte_hast_komp_sommer_max_temp with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_a_alm with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_b_alm with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_c_alm with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_filter_wrn_alm with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_filter_alm with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_tilluftstemp_lav_alm with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_filter_wrn_cnt_alm with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_tillufts_temperatur with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_inntaks_temperatur with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_avtrekks_temperatur with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_overhetingstemperatur_tilluft with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_relativ_fukt_avtrekk with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_tilluft_sp with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_avtrekksluft_sp with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_avtrekksluft_min_sp with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_avtrekksluft_max_sp with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_eco_varme_offset with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_tilluftsvifte with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_avtrekksvifte with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_el_kolbe_padrag with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_el_kolbe_av_pa with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_varmegjenvinner with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_komfyravtrekk_trykkvakt_av_pa with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_modus_status_register with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_tilluft_rom_avtrekk_reg with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_manuell_modus_kommando_register with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_time_year with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_time_md with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_time_day with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_time_hour with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_time_min with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_time_sec with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_sch_active_temp_offset with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_sch_inactive_temp_offset with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_sch_prd1_day1_start_h with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_sch_prd1_day1_start_m with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_sch_prd1_day1_end_h with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_sch_prd1_day1_end_m with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_kalkulert_fukt_avtrekk with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_kalkulert_fukt_inntak with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_sommerdrift_vinterdrift with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_filterbytte_alm with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_filterbytte_md_intervall with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_tid_til_filter_bytte_s with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_nedtelling_modus_tid_s with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_ferie_modus_varighet with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_borte_modus_varighet with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_ildsted_modus_varighet with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_boost_modus_varighet with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_party_modus_varighet with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_fuktoverforing_sp with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_vifte_hast_komp_avlest with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_vifte_hast_komp_vinter with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_vifte_hast_komp_vinter_start_temp with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_vifte_hast_komp_vinter_max_temp with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_vifte_hast_komp_sjekkes with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_vifte_hast_komp_sommer with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_vifte_hast_komp_sommer_start_temp with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_vifte_hast_komp_sommer_max_temp with int is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_a_alm with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_b_alm with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_c_alm with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_filter_wrn_alm with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_filter_alm with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_tilluftstemp_lav_alm with uint is not valid, trying to convert 2021-09-04 14:37:27 WARNING (MainThread) [homeassistant.components.modbus.validators] vtr250_filter_wrn_cnt_alm with uint is not valid, trying to convert 2021-09-04 14:37:56 ERROR (SyncWorker_3) [homeassistant.components.modbus.modbus] Pymodbus: VTR250: Exception Response(131, 3, IllegalAddress) 2021-09-04 14:37:56 DEBUG (SyncWorker_5) [homeassistant.components.modbus.modbus] Pymodbus: VTR250: Exception Response(131, 3, IllegalAddress) 2021-09-04 14:37:56 DEBUG (SyncWorker_1) [homeassistant.components.modbus.modbus] Pymodbus: VTR250: Exception Response(131, 3, IllegalAddress) 2021-09-04 14:38:13 ERROR (SyncWorker_2) [homeassistant.components.modbus.modbus] Pymodbus: VTR250: Exception Response(131, 3, IllegalAddress) 2021-09-04 14:38:13 DEBUG (SyncWorker_4) [homeassistant.components.modbus.modbus] Pymodbus: VTR250: Exception Response(131, 3, IllegalAddress) 2021-09-04 14:38:13 DEBUG (SyncWorker_6) [homeassistant.components.modbus.modbus] Pymodbus: VTR250: Exception Response(131, 3, IllegalAddress) 2021-09-04 14:38:14 DEBUG (SyncWorker_0) [homeassistant.components.modbus.modbus] Pymodbus: VTR250: Exception Response(134, 6, GatewayNoResponse) 2021-09-04 14:38:28 ERROR (SyncWorker_1) [homeassistant.components.modbus.modbus] Pymodbus: VTR250: Exception Response(131, 3, IllegalAddress) 2021-09-04 14:38:28 DEBUG (SyncWorker_6) [homeassistant.components.modbus.modbus] Pymodbus: VTR250: Exception Response(131, 3, IllegalAddress) 2021-09-04 14:38:28 DEBUG (SyncWorker_3) [homeassistant.components.modbus.modbus] Pymodbus: VTR250: Exception Response(131, 3, IllegalAddress)

prilly-dev commented 2 years ago 
2021-09-04 14:46:08 DEBUG (SyncWorker_2) [pymodbus.client.sync] New Transaction state 'SENDING'
2021-09-04 14:46:08 DEBUG (SyncWorker_0) [pymodbus.client.sync] New Transaction state 'SENDING'
2021-09-04 14:46:08 DEBUG (SyncWorker_3) [pymodbus.client.sync] New Transaction state 'SENDING'
2021-09-04 14:46:09 DEBUG (SyncWorker_5) [pymodbus.client.sync] New Transaction state 'SENDING'
2021-09-04 14:46:09 ERROR (SyncWorker_5) [homeassistant.components.modbus.modbus] Pymodbus: VTR250: Exception Response(131, 3, IllegalAddress)
2021-09-04 14:46:09 DEBUG (SyncWorker_1) [pymodbus.client.sync] New Transaction state 'SENDING'
2021-09-04 14:46:09 DEBUG (SyncWorker_4) [pymodbus.client.sync] New Transaction state 'SENDING'
2021-09-04 14:46:09 DEBUG (SyncWorker_7) [pymodbus.client.sync] New Transaction state 'SENDING'
2021-09-04 14:46:09 DEBUG (SyncWorker_6) [pymodbus.client.sync] New Transaction state 'SENDING'
2021-09-04 14:46:10 DEBUG (SyncWorker_2) [pymodbus.client.sync] New Transaction state 'SENDING'
2021-09-04 14:46:10 DEBUG (SyncWorker_0) [pymodbus.client.sync] New Transaction state 'SENDING'
2021-09-04 14:46:10 DEBUG (SyncWorker_3) [pymodbus.client.sync] New Transaction state 'SENDING'
2021-09-04 14:46:10 DEBUG (SyncWorker_5) [pymodbus.client.sync] New Transaction state 'SENDING'
2021-09-04 14:46:11 DEBUG (SyncWorker_1) [pymodbus.client.sync] New Transaction state 'SENDING'

It seems that adding slave: 1 actualy fixed the problem, i guess slave 1 is added to the address hence illegalAddress

prilly-dev commented 2 years ago

It is working, but its extremly slow, and its kind of hit and miss if switch is accepting the input, sometime it just goes back to initial value, (if trying to turn on it goes to on, and then it just goes back to off by itself)

janiversen commented 2 years ago

Solve the convert problems int -> int16/int32 etc. that will reduce the noise in the log. When you switch we read the status imidiatly so that cannot be the problem.

Maybe you have something else “eating” your CPU.

janiversen commented 2 years ago

Closing issue as solved. Switch hit and miss in accepting input is not likely a modbus problem, but a problem somewhere else.