edwork / homeassistant-peloton-sensor

A platform which allows you to get current and past ride data from Peloton into HomeAssistant
Apache License 2.0
78 stars 16 forks source link

Heartrate #44

Closed Bubbalax closed 1 year ago

Bubbalax commented 2 years ago

I created automations based on current heartrate with the old integration. This does not seem possible with the current integration. I only see Avg HR and Max HR. Am I missing something?

BTW - Great work on this integration. Thank you!

edwork commented 2 years ago

Hello!

I haven't lost track of this, at a quick glance I think I see the current heart rate is still populated in my latest workout overview. I'll try and add it back shortly.

Thank you for the feedback and for enjoying the integration! -Ed

tamu20 commented 2 years ago

+1 on this request. Most of my automations are based on the previous 'current heartrate' sensor.

Thanks again for your time on this integration.

Bubbalax commented 1 year ago

Hi Ed. I am circling back on this one. I am not trying to push but curious if still plan on addressing this. Thank for you the latest release!

Miablo commented 1 year ago

@Bubbalax @tamu20 i just got some code merged into master that should bring back this functionality 🤞

Bubbalax commented 1 year ago

@Miablo - Any way I can manually update the change? Its Winter and I would like to play with it! :) Thank you!

edwork commented 1 year ago

Version 0.9.0 has been released and now fully supports theses values. Feel free to open this issue back up if you run into anything unexpected.

Thanks!