Skip to content

Add mapping for POOLDOSE DOUBLE SPA (PDPR1H04AW100_FW539292)#25

Merged
lmaertin merged 4 commits intolmaertin:mainfrom
ronaldvdmeer:feat/add-mapping-PDPR1H04AW100-FW539292
Mar 15, 2026
Merged

Add mapping for POOLDOSE DOUBLE SPA (PDPR1H04AW100_FW539292)#25
lmaertin merged 4 commits intolmaertin:mainfrom
ronaldvdmeer:feat/add-mapping-PDPR1H04AW100-FW539292

Conversation

@ronaldvdmeer
Copy link
Contributor

@ronaldvdmeer ronaldvdmeer commented Mar 11, 2026

Summary

Adds device mapping file for the SEKO POOLDOSE DOUBLE SPA (PDPR1H04AW100, firmware 539292).

Closes #20

Device details

  • Product code: PDPR1H04AW100
  • Firmware code: 539292
  • Model name: POOLDOSE DOUBLE SPA
  • Gateway firmware: 539156 (release 3.30)

Mapping contents

This is a significantly more capable device than the single pH+ORP models, with 65 entities total:

Type Count Details
sensor 26 pH, ORP, CL, temperature, dosing types, calibration, OFA times, delays, config
binary_sensor 23 Standard alarms + OFA2 (pH/ORP/CL), water temp, concentration, system standby
number 11 pH/ORP/CL setpoints, dosing timers (on/off), power-on & flow delay timers
switch 3 Pause dosing, pump monitoring, frequency input
select 1 Water meter unit (m³/L)

New entities compared to PDPR1H1HAW100

  • CL dosing: CL target setpoint, peristaltic CL dosing mode, time on/off CL dosing
  • CL calibration: CL calibration type, offset, slope
  • Extended alarms: OFA2 for pH/ORP/CL, water too cold/hot, pH/CL concentration alerts, system standby
  • Delay timers: Power-on delay timer + status, flow delay timer + status
  • Configuration: Device config (pH/ORP vs pH/ORP/Chlorine mode), temperature unit

Testing

  • Added 8 tests: mapping loading, sensor/binary_sensor/number/switch/select verification, entity counts
  • All 131 tests pass

Data source

Mapping created from debuginfo.json, instantvalues.json and strings.json provided in issue #20 .

Add device mapping file for SEKO POOLDOSE DOUBLE SPA with product code
PDPR1H04AW100 and firmware code 539292.

This is a significantly more capable device than the single pH+ORP
models, featuring:
- pH, ORP and Chlorine measurement sensors
- CL dosing controls (setpoint, type, peristaltic, time on/off)
- CL calibration sensors (type, offset, slope)
- Extended alarm binary sensors (OFA2 for pH/ORP/CL, water
  temperature, concentration, system standby)
- Delay timers (power-on and flow delay with enable/disable)
- Device configuration sensor (pH/ORP vs pH/ORP/Chlorine mode)
- Temperature unit selection

Entity counts: 26 sensors, 23 binary_sensors, 11 numbers,
3 switches, 1 select (65 total).

Mapping created from debug data provided in issue report.
Added 8 tests validating loading, entity types and counts.

All 131 tests pass.
@ronaldvdmeer
Copy link
Contributor Author

Awaiting response to test the fix suggested in #20

@lmaertin
Copy link
Owner

I propose to take this into the next release.

If the affected user does not respond in next 7 days, let's proceed.
I would mark the support for the model as experimental in the readme than.

@ronaldvdmeer
Copy link
Contributor Author

ronaldvdmeer commented Mar 14, 2026

I propose to take this into the next release.

I agree, that approach makes sense to me.
I plan to convert the PR from draft to ready for review on the 19th.

@ronaldvdmeer
Copy link
Contributor Author

Since there has been no response from the affected user so far (#20), I'll move this PR from draft to ready for review.

@ronaldvdmeer ronaldvdmeer marked this pull request as ready for review March 15, 2026 13:57
@lmaertin
Copy link
Owner

wrt. discussion in #21 , the enabled/disable sensors shall be mapped to True/False and defined as binary_sensors

@ronaldvdmeer
Copy link
Contributor Author

Convert circulation_pump_status, power_on_delay_status, and flow_delay_status from sensor type with string conversion (disabled/enabled) to binary_sensor type with boolean conversion (false/true).

Changes

  • model_PDPR1H04AW100_FW539292.json: Changed type from sensor to binary_sensor and conversion values from "disabled"/"enabled" to false/true for:
    • circulation_pump_status
    • power_on_delay_status
    • flow_delay_status
  • test_mapping_info.py: Updated entity type counts (sensor 26→23, binary_sensor 23→26) and moved the 3 entities from sensor to binary_sensor test assertions

All 144 tests pass.

@lmaertin lmaertin merged commit abfc5e0 into lmaertin:main Mar 15, 2026
9 checks passed
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

Successfully merging this pull request may close these issues.

Missing mapping file causes setup failure in HA (PDPR1H04AW100 / FW539292)

2 participants