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

leshow.heater.bs1 shows heater internal temperature as current temperature #316

Closed
mouth4war opened this issue Dec 20, 2021 · 8 comments
Closed

Comments

@mouth4war
Copy link

https://miot-spec.org/miot-spec-v2/instance?type=urn:miot-spec-v2:device:heater:0000A01A:leshow-bs1:1

Room temp is showing 40C (heater fins temperature) instead of room environment temperature (which is showing as a sensor). These variables should be switched.

@mouth4war
Copy link
Author

image

Fix:
Replace Environment Temperature with Heater Temperature in sensor name.
Switch current temperature and heater temperature around

@al-one
Copy link
Owner

al-one commented Dec 21, 2021

I don't think there's anything wrong with it.

@mouth4war
Copy link
Author

My room is not currently at 36C, it is at 25C which is showing as a sensor instead of climate current temperature.

36C is the heater internal temperature and should instead be shown as a sensor called "Heater Internal Temperature".

25C is the room current temperature and should be shown as climate current temperature.

@mouth4war mouth4war changed the title leshow.heater.bs1 shows incorrect room temperature leshow.heater.bs1 shows heater internal temperature as current temperature Dec 23, 2021
@mtbhomer
Copy link

mtbhomer commented Jan 7, 2023

Hello, I also encountered this problem with the wrong temperature (internal instead of environment temperature). I'm curious if there could be a fix for this? @al-one @mouth4war

Screenshot 2023-01-07 at 12 20 30

Screenshot 2023-01-07 at 12 21 11

@al-one
Copy link
Owner

al-one commented Jan 8, 2023

Show me the entity attributes.

@al-one
Copy link
Owner

al-one commented Jan 8, 2023

Try master branch.

@mtbhomer
Copy link

mtbhomer commented Jan 8, 2023

Show me the entity attributes.

Here you go:

hvac_modes:
  - auto
  - 'off'
min_temp: 18
max_temp: 28
target_temp_step: 1
preset_modes:
  - Sleep
  - 'off'
current_temperature: 24
temperature: 22
hvac_action: heating
preset_mode: null
model: leshow.heater.bs1
lan_ip: 192.168.31.165
mac_address: redacted
entity_class: MiotClimateEntity
home_room: redacted
miot_type: urn:miot-spec-v2:device:heater:0000A01A:leshow-bs1:1
heater.on: true
heater.mode: 0
heater.target_temperature: 22
heater.temperature: 41
alarm: false
physical_controls_locked: false
environment.temperature: 24
countdown.countdown_time: 0
state_updater: cloud
sub_entities:
  - heater-2.on-1
  - countdown-3.countdown_time-1
  - environment-4.temperature-7
  - physical_controls_locked-5.physical_controls_locked-1
icon: mdi:heat-wave
friendly_name: Baseboard Heater
supported_features: 17

@mtbhomer
Copy link

mtbhomer commented Jan 8, 2023

Try master branch.

Awesome! Just tested the changes, and happy to report it's now showing the correct environment temperature attribute. Tks!

@al-one al-one closed this as completed Jan 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants