Skip to content

Add Daikin64 AC Remote app#1099

Open
mrcalifornium wants to merge 5 commits into
flipperdevices:mainfrom
mrcalifornium:mrcalifornium/daikin64_ac_remote_0.1
Open

Add Daikin64 AC Remote app#1099
mrcalifornium wants to merge 5 commits into
flipperdevices:mainfrom
mrcalifornium:mrcalifornium/daikin64_ac_remote_0.1

Conversation

@mrcalifornium

@mrcalifornium mrcalifornium commented Jun 21, 2026

Copy link
Copy Markdown

Application Submission

Daikin64 APGS02-compatible AC remote for Flipper Zero. It dynamically generates Daikin64 HVAC IR packets for power toggle, mode, temperature, fan speed, swing, turbo, quiet, sleep, timers, and five favorite presets.

Extra Requirements

Requires a Daikin indoor unit compatible with the Daikin64/APGS02 IR protocol. No extra hardware is required beyond the Flipper Zero IR transmitter.

Author Checklist

  • I've read the contribution guidelines and my PR follows them
  • I own the code I'm submitting or have code owner's permission to submit it
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have validated the manifest file(s) with python3 tools/bundle.py --nolint applications/CATEGORY/APPID/manifest.yml bundle.zip

AI usage disclosure

  • Partially AI assisted (clarify below which code was AI assisted and briefly explain what it does).

OpenAI Codex assisted with porting and structuring the Flipper app, UI iteration, README/catalog metadata, and Daikin64 protocol implementation. Claude Code assisted with the release audit, lint/format fixes, and protocol test corrections. The protocol behavior is based on IRremoteESP8266's IRDaikin64 implementation plus APGS02 captures and on-device testing with a real Flipper Zero and Daikin APGS02 remote.

Reviewer Checklist (Don't fill this out!)

  • Bundle is valid
  • There are no obvious issues with the source code
  • I've ran this application and verified its functionality

@mrcalifornium mrcalifornium force-pushed the mrcalifornium/daikin64_ac_remote_0.1 branch from 45ef873 to 25a354c Compare June 21, 2026 20:43

@xMasterX xMasterX left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hi, you have two application.fam files and very strange source placement in the repo in applications_user folder
Please fix that, there should be only one application.fam file, if your app is fully located in subfolder in the repo you should use subdir: "path/123/app" in location section in manifest here

Also please return Reviewer Checklist you removed from PR description template

@xMasterX xMasterX added the fixes needed Unresolved issues with the app requiring maintainer's attention label Jun 30, 2026
@mrcalifornium

Copy link
Copy Markdown
Author

Addressed this review feedback:

  • Removed the root application.fam; the source repo now has only applications_user/daikin64_ac_remote/application.fam.
  • Moved the Daikin64 protocol library and catalog assets under applications_user/daikin64_ac_remote/ so the app is self-contained in that folder.
  • Updated this catalog manifest to use location.subdir: applications_user/daikin64_ac_remote.
  • Re-ran local catalog validation successfully:
    python tools/bundle.py --nolint applications/Infrared/daikin64_ac_remote/manifest.yml bundle.zip

Points the manifest at the commit that passes catalog CI lint and
removes the power-toggle clock skew.
@mrcalifornium

Copy link
Copy Markdown
Author

The source repo now has a single application.fam with the app self-contained under applications_user/daikin64_ac_remote, and the manifest uses location.subdir pointing there. Manifest now pins commit 64314c6, which also fixes clang-format lint (validated locally with the full tools/bundle.py run incl. lint against the rc SDK).

@mrcalifornium mrcalifornium requested a review from xMasterX July 5, 2026 06:52
@xMasterX

Copy link
Copy Markdown
Collaborator

Hi, good

One problem left, the screenshots are not clean 2-color qFlipper shots (around 49 colors). qFlipper screenshot
gives exactly orange + black at 512x256. Can you please take them again from qFlipper without any
editing? Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

app (new) fixes needed Unresolved issues with the app requiring maintainer's attention

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants