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

tempdiffcool range parameter in JSON wrong #1215

Closed
basalblas opened this issue Jul 10, 2023 · 2 comments
Closed

tempdiffcool range parameter in JSON wrong #1215

basalblas opened this issue Jul 10, 2023 · 2 comments
Labels
bug Something isn't working
Milestone

Comments

@basalblas
Copy link

The current JSON for tempdiffcool is producing the error below in HA log:

ERROR (MainThread) [homeassistant.components.mqtt.number] Invalid value for number.boiler_temp_diff_tc3_tc0_cool: 2.0 (range 3.0 - 10.0)

The latest recommended value for this setting on a Nefit EnviLine All-Electric Split is 2K as can be found here

The JSON at http://ems-esp/api/boiler/tempdiffcool defines the range to be min 3 to max 10, and should be changed to min 2 max 10.

{
  "name": "tempdiffcool",
  "fullname": "Temp.vers. TC3/TC0 koel.",
  "circuit": "",
  "value": 2.0,
  "type": "number",
  "min": 3,
  "max": 10,
  "uom": "K",
  "readable": true,
  "writeable": true,
  "visible": true
}

@proddy proddy added this to the v3.6.0 milestone Jul 10, 2023
@proddy proddy added the bug Something isn't working label Jul 10, 2023
@proddy
Copy link
Contributor

proddy commented Jul 10, 2023

Thanks for reporting - we'll change in the dev version

MichaelDvP added a commit to MichaelDvP/EMS-ESP32 that referenced this issue Jul 11, 2023
@proddy
Copy link
Contributor

proddy commented Jul 11, 2023

will be in dev2 when Michael merges his PR

@proddy proddy closed this as completed Jul 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants