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

Fix Auxiliary Heater for ICE, add outside_temperature #260

Merged
merged 2 commits into from
Dec 3, 2024

Conversation

prvakt
Copy link
Collaborator

@prvakt prvakt commented Dec 3, 2024

There is a difference between how state from auxiliary heater can be obtained between ICE and PHEV

  • PHEV cars are not using auxiliary-heating endpoint at all and state is available via air-conditioning
  • ICE cars are using both endpoints, but state inside air-conditioning is always state="INVALID" independent on the real state. Because of that, we must retrieve state from auxiliary-heating endpoint. Because of that, auxiliary-heating data have been added to the Vehicle class and can be retrieved via get_vehicle()
  • added outside_temperature as it can be retrieved from auxiliary-heating endpoint if car with AUXILIARY_HEATING and OUTSIDE_TEMPERATURE capability is present
  • added AUXILIARY_HEATING_BASIC capability

Tomas Prvak added 2 commits December 3, 2024 10:15
added AUXILIARY_HEATING_BASIC capability
added targetTemperature to AuxiliaryHeating class
added auxiliary_heating object to vehicle
added request for auxiliary_heating into get_vehicle()
@WebSpider WebSpider added bug Something isn't working enhancement New feature or request labels Dec 3, 2024
Copy link
Contributor

@WebSpider WebSpider left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your continued work on the heater! 💪

myskoda/myskoda.py Show resolved Hide resolved
@WebSpider WebSpider changed the title auxiliary-heating improvement Fix Auxiliary Heater for ICE, add outside_temperature Dec 3, 2024
@WebSpider WebSpider merged commit f81c032 into skodaconnect:main Dec 3, 2024
1 of 2 checks passed
@prvakt prvakt deleted the auxiliary-heating branch December 4, 2024 17:45
zaptm pushed a commit to zaptm/myskoda that referenced this pull request Dec 8, 2024
* added AUXILIARY_HEATING_BASIC capability
* added targetTemperature to AuxiliaryHeating class
* added auxiliary_heating object to vehicle
* added request for auxiliary_heating into get_vehicle()
* fixtures updated

---------

Co-authored-by: Tomas Prvak <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants