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

feat: Start Autoconf Framework #19

Merged
merged 5 commits into from
Apr 2, 2024

Conversation

KazWolfe
Copy link
Contributor

Pulled out of the PR #17 for maintainer sanity, and starts work on Sammy1Am/temp-issues#9.

  • Fix a bug where the auto fan flag wasn't properly inverted.
  • Move supported fan speeds to the .cpp file for packets.
  • Add skeleton packet.asTraits() to quickly get core heatpump-supported features.
    • Not to be used directly, as any integration may still build their own custom traits.
  • Always request extended connect information as part of boot cycle, and cache the results.

@KazWolfe KazWolfe force-pushed the feat/autoconf-framework branch from 0a1063f to 6f9e108 Compare March 31, 2024 23:14
components/mitsubishi_uart/muart_packet.h Outdated Show resolved Hide resolved
components/mitsubishi_uart/mitsubishi_uart.cpp Outdated Show resolved Hide resolved
@KazWolfe KazWolfe force-pushed the feat/autoconf-framework branch from 6f9e108 to e26f653 Compare April 1, 2024 06:27
KazWolfe added 4 commits April 1, 2024 21:46
- Fix auto fan speed flag inversion
- Add temporary HVaneSwing control field
- Move getSupportedFanSpeeds to .cpp file
- Add core traits converter
- Add new _capabilitiesRequested field to track if we've asked for capabilities yet (in a non-blocking way).
- Tie HVaneSwing support to regular vane swing.
@KazWolfe KazWolfe force-pushed the feat/autoconf-framework branch from e26f653 to f4f5d11 Compare April 2, 2024 04:47
@Sammy1Am Sammy1Am merged commit b0906a8 into Sammy1Am:main Apr 2, 2024
@KazWolfe KazWolfe deleted the feat/autoconf-framework branch April 8, 2024 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants