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.

Sign up for GitHub

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

Jump to bottom

Update rotenso_ronix_heatpump.yaml #2145

Merged
merged 1 commit into from
Jul 22, 2024
Merged

Update rotenso_ronix_heatpump.yaml #2145

merged 1 commit into from
Jul 22, 2024

Conversation

piotrek2555
Copy link
Contributor

Change range temperature from 16 to 31

@piotrek2555
Update rotenso_ronix_heatpump.yaml
3de4a07 
Change range temperature from 16 to 31
@make-all
Copy link
Owner

make-all commented Jul 22, 2024 •
edited
Loading

@cykus, as you originally submitted this device, do you have any comments on this change? Was it just copy-paste from another device which maybe works as a water heater or supports both C and F, so would go up to that range?

@piotrek2555
Copy link
Contributor Author

I only copy and paste from other device. In my app smart I have only to 31 degrees celsius, but I not sure or device support Fahrenheit temperature.

@make-all
Copy link
Owner

make-all commented Jul 22, 2024 •
edited
Loading

Thank you for confirming Are both accounts you then?

@make-all make-all merged commit 42f0e9f into make-all:main Jul 22, 2024
4 checks passed
@cykus
Copy link
Contributor

cykus commented Jul 22, 2024 •
edited
Loading

@make-all i set it to 88 because of debug info on iot tuya, as @piotrek2555 said in app i can set it only to 31c
temp_set | Integer | { "unit": "℃", "min": 160, "max": 880, "scale": 1, "step": 5 }

on debug info i see second dps for Fahrenheit, but i don't know have tu use two set_temps
temp_set_f | Integer | { "unit": "℉", "min": 61, "max": 88, "scale": 0, "step": 1 }

i dont't see temperature_unit dps

make-all added a commit that referenced this pull request Jul 28, 2024
@make-all
Docs for recent merged PRs
5f09d86 
- rename human_presence_sensor.yaml to more specific name. There is no
  evidence that is is generic enough to go without some identifier.

PR #2153, #2150, #2146, #2145, #2143, #2137, #2136, #2134, #2116,
PR #2087, #2033
timlaing pushed a commit to timlaing/tuya-local that referenced this pull request Aug 8, 2024
@make-all @timlaing
Docs for recent merged PRs
30f0120 
- rename human_presence_sensor.yaml to more specific name. There is no
  evidence that is is generic enough to go without some identifier.

PR make-all#2153, make-all#2150, make-all#2146, make-all#2145, make-all#2143, make-all#2137, make-all#2136, make-all#2134, make-all#2116,
PR make-all#2087, make-all#2033
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
Tuya Local
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@piotrek2555 @make-all @cykus