search

sinricpro / non-sdk-issues

Report non sdk related issues here (Alexa, Google Home, SmartThings, IFTTT, API)
2 stars 0 forks source link

Alexa does not set Thermostat temperature ESP32 #21

Closed JulianGold closed 3 years ago

JulianGold commented 3 years ago

Setting temperature from Alexa results in “sorry, device is not responding”. Setting a temperature increment or turning thermostat on or off works. Temperature setting in Alexa is in Fahrenheit.

kakopappa commented 3 years ago
  1. Do you see any errors in the activity log?
  2. are you returning true after handling the set onTargetTemperature? https://github.com/sinricpro/esp8266-esp32-sdk/blob/master/examples/Thermostat/Thermostat.ino#L53
JulianGold commented 3 years ago
  1. There are no errors in the log. The onTargetTemperature function is not called
  2. Yes, I am returning true in the onTargetTemperature function. However, the function is not triggered.

The other "events" such as increasing or decreasing the temperature work well. Setting the temperature to a specified level however does not work from Alexa. It does work from the SinricPro mobile app but it does not work from Alexa.

sivar2311 commented 3 years ago

Does this happen with unmodified example sketch? Which SDK Version do you use? Do you see incoming requests when debugging is enabled?

Edit: Did you attach the callback to the device ?

JulianGold commented 3 years ago

The issue occurs with the example sketch. I am using SDK 2.9.1. The issue occurred in the past, at least since 2.8.1. I did not test with debug enabled yet. The callback is attached.

The issue might be due to Alexa working with Fahrenheit vs Celsius but I'm not sure.

sivar2311 commented 3 years ago

Does it get triggered when using portal.sinric.pro ? Which scale did you set in your SinricPro account?

sivar2311 commented 3 years ago

Ok, i was able to reproduce the issue - we will check this.

There seems to be an issue with Fahrenheit.

sivar2311 commented 3 years ago

Can you tell me where exactly the error occurs?

JulianGold commented 3 years ago

Alexa app as well as Alexa voice input. Both have this issue. Alexa is set to Fahrenheit.

SinricPro portal works well and it triggers the callback. SinricPro scale is set to Celsius. If I set the scale to Fahrenheit the portal still displays the current temperature in Celsius and is acting weird displaying the temperature in Celsius but then saying it's Fahrenheit (the number for the temperature is followed by the word "Fahrenheit").

sivar2311 commented 3 years ago

Fixed in 2.9.2 ✔️ Please update and check again.

JulianGold commented 3 years ago

Retested and it's a bit better. The message is received and callback triggered. The temperature received in the message however is always 17.8888 degrees lower than it should be. For example if I ask Alexa to set the temperature to 70, the message received at micro-controller device says "52.22222"

sivar2311 commented 3 years ago

Yeah, i had the same issue and thought it is related to my DE settings. Issue in sdk is fixed. @kakopappa can you check whats going on ?

kakopappa commented 3 years ago

Hi @JulianGold

There was an issue on the server-side and I have fixed it. Please give it a go when you get a chance

JulianGold commented 3 years ago

👍 Works great now. Thank you!

JulianGold commented 3 years ago

Perhaps less important (for me) but the SinricPro iOS App is still showing the target temperature off by 8 degrees.

kakopappa commented 3 years ago

Is the portal showing the correct value?

On Sat, 20 Mar 2021 at 9:53 PM Julian Gold @.***> wrote:

Perhaps less important (for me) but the SinricPro iOS App is still showing the target temperature off by 8 degrees.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/sinricpro/esp8266-esp32-sdk/issues/161#issuecomment-803369690, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABZAZZTF3WY2FC3FG76BNUDTESZHRANCNFSM4ZFVTD5Q .

JulianGold commented 3 years ago

Yes, the portal shows the right temperature now. The iOS app is the one that is not showing the right target temperature.

kakopappa commented 3 years ago

Hello @JulianGold

Thank you for letting us know. I will look into this and update this issue again when it's fixed.

I going to move this issue out of the sdk repo to non-sdk repo since it's no longer an SDK issue.

kakopappa commented 3 years ago

Hi,

New version is available with the fix 2.10.1 can you give it a try when available

On Sun, 21 Mar 2021 at 10:46 AM Julian Gold @.***> wrote:

Setting temperature from Alexa results in “sorry, device is not responding”. Setting a temperature increment or turning thermostat on or off works. Temperature setting in Alexa is in Fahrenheit.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/sinricpro/non-sdk-issues/issues/21, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABZAZZRJJCAJYJNRJDR7R4TTEVTYZANCNFSM4ZRDSTGQ .

stale[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.