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

Misleading name of exhaustAirTemperatureBeforeHeatRecovery reading #122

Closed
ajulk7 opened this issue Nov 23, 2024 · 3 comments
Closed

Misleading name of exhaustAirTemperatureBeforeHeatRecovery reading #122

ajulk7 opened this issue Nov 23, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@ajulk7
Copy link

ajulk7 commented Nov 23, 2024

I've been testing eda-modbus-bridge with "LTR-6 eco EDW" model (EDA version is 2.14), which has water circulating radiator as after heater. Works nicely with LTR-6, except that return water temperature is not available in /summary. That's an interesting reading for EDW owners.

But it seems register value 3x0011 which is shown in summary with name "exhaustAirTemperatureBeforeHeatRecovery" is actually same value as the 3x0012 returning water value. I tried this by modifying the code to show 3x0012 also. There is same value in both registers. Control panel also shows the same reading as "Paluuvesi" (returning water).

In the Enervent registry listing, it says "X8" for both. This is a bit weird design from Enervent.

image

Electric diagram in the EDA manual says that X8 sensor is return water temperature sensor. And that the X8 sensor is only available in EDW models. I wonder what's the 3x0011 reading value for ED/EDE?

image

I think it would make sense to change the reading name "exhaustAirTemperatureBeforeHeatRecovery" to "returnWaterTemperature", to avoid confusion.

@ajulk7
Copy link
Author

ajulk7 commented Nov 24, 2024

Did more reading at the Enervent docs site. In PELICAN/PEGASOS eco PRO greenair HP (HP_2012_2_en.pdf) there is difference use for X8, "After coil exhaust air sensor" (in finnish version it's "poistoilmapatterin jälkeinen anturi"). But this does not match the name in registry listing either.

image

Maybe it could be conditional, if device is EDW then show 3x0012 "returnWaterTemperature", else show 3x0011 "exhaustAirTemperatureBeforeHeatRecovery". Or just include both readings.

@Jalle19
Copy link
Owner

Jalle19 commented Nov 24, 2024

Interesting find!

I think the best option would like you said be to make the current exhaustAirTemperatureBeforeHeatRecovery sensor optional and only enabled on ED/EDE models, then introduce another optional sensor returnWaterTemperature for EDW models.

@Jalle19 Jalle19 added the enhancement New feature or request label Nov 24, 2024
@Jalle19
Copy link
Owner

Jalle19 commented Nov 24, 2024

In the eWind register list, 3x0012 is mentioned as:

Holding reg 12 | HREG_T_WR | Return water | Return water temperature in supply air heater (sensor TE45)

Jalle19 added a commit that referenced this issue Nov 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants