Skip to content

tidy A/C controller use#663

Open
nmschulte wants to merge 3 commits intoFOME-Tech:masterfrom
nmschulte:nms/tidy-ac-controller
Open

tidy A/C controller use#663
nmschulte wants to merge 3 commits intoFOME-Tech:masterfrom
nmschulte:nms/tidy-ac-controller

Conversation

@nmschulte
Copy link
Contributor

@nmschulte nmschulte commented Sep 3, 2025

more of #623; #660 (comment)

void onSlowCallback() override;

virtual bool isAcEnabled() const;
virtual bool isAcCompressorEnabled() const;
Copy link
Contributor Author

@nmschulte nmschulte Sep 3, 2025

Choose a reason for hiding this comment

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

getAcCompressorState? this isn't actually used as acRelayAlternatorDutyAdder is going away

@nmschulte nmschulte requested a review from mck1117 September 3, 2025 22:24
it accounts for all things A/C actuation, post switch demand (but pre-delay)
@nmschulte nmschulte force-pushed the nms/tidy-ac-controller branch from 662af78 to 5552767 Compare September 3, 2025 22:49
@nmschulte nmschulte added the 💰 tech debt 💰 Not causing an active problem, but good to fix. label Sep 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💰 tech debt 💰 Not causing an active problem, but good to fix.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant