Repository navigation
Read climatisation from HVAC_SUMMARY; quieter retries (0.5.1) - #8
Merged
Merged
Conversation
Porsche no longer reports remote climatisation through CLIMATIZER_STATE (the measurement comes back empty), so has_remote_climatisation was False on every car and climatisation could not be used. Same change as CJNE#98: - request HVAC_SUMMARY instead of CLIMATIZER_STATE; - has_remote_climatisation / remote_climatise_on read HVAC_SUMMARY (on = status other than OFF), new seat_heating property with the climate zones the car reports; - REMOTE_CLIMATIZER_START payload carries "spin": null; - user agent of the Porsche app, which the API needs to wake the car up on currentoverview. Retries of transient errors (429/5xx, transport) are logged at debug level: the caller already reports the final failure, and a warning per attempt flooded the Home Assistant log. Tests also run on Python 3.14. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Porsche no longer reports remote climatisation through
CLIMATIZER_STATE(the measurement comes back empty), sohas_remote_climatisationwasFalseon every car. Same change as CJNE#98:HVAC_SUMMARYinstead ofCLIMATIZER_STATE;has_remote_climatisation/remote_climatise_onread it (on = status other thanOFF); newseat_heatingproperty with the climate zones the car reports;REMOTE_CLIMATIZER_STARTpayload carries"spin": null;currentoverview.Retries of transient errors are logged at debug level: the caller already reports the final failure, and a warning per attempt flooded the Home Assistant log.
Tests: 104 pass on Python 3.12 and 3.14 (CI matrix now includes 3.14), coverage 81.9 %. Not tested on a car with remote climatisation (the test car has none).
🤖 Generated with Claude Code