Skip to content

Fix TCI audio corruption and add IQ stream support for TCI Remote (#1182)#1247

Closed
aethersdr-agent[bot] wants to merge 1 commit intomainfrom
aetherclaude/issue-1182
Closed

Fix TCI audio corruption and add IQ stream support for TCI Remote (#1182)#1247
aethersdr-agent[bot] wants to merge 1 commit intomainfrom
aetherclaude/issue-1182

Conversation

@aethersdr-agent
Copy link
Copy Markdown
Contributor

Summary

Fixes #1182


Generated by AetherClaude (automated agent for AetherSDR)

)

Three fixes for TCI Remote Android app compatibility:

1. Fix audio_stream_sample_type parsing: string values like "float32"
   were parsed as 0 (int16) by toInt(), causing format mismatch and
   garbled audio. Now handles both numeric and string format names.

2. Fix TCI binary frame header length field: was set to total sample
   count (frames * channels) instead of per-channel count (frames),
   per TCI v2.0 spec. This caused clients to misparse audio payloads.

3. Add IQ data forwarding over TCI WebSocket: iq_start/iq_stop now
   track per-client IQ state, and DAX IQ VITA-49 packets are
   byte-swapped and sent as type=0 binary frames, enabling the
   panadapter/waterfall display in TCI Remote.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@aethersdr-agent aethersdr-agent bot mentioned this pull request Apr 13, 2026
7 tasks
@aethersdr-agent
Copy link
Copy Markdown
Contributor Author

CI Failure Analysis

TL;DR: This is not a problem with your code. The build failure is caused by a missing cjson/cJSON.h header that was fixed on main after this branch was created. A rebase will resolve it.

What failed

Both the build and analyze (cpp) jobs failed at the CMake build step with:

third_party/mosquitto/include/mosquitto/broker_control.h:31:10: fatal error: cjson/cJSON.h: No such file or directory

Why it failed

This PR branch (aetherclaude/issue-1182) is based on commit f27b7f1, but the fix for the bundled mosquitto cJSON stub header landed on main afterward in commit 517c4a2 ("Fix CI: stub cJSON header in bundled mosquitto"). Since the branch predates that fix, the CI build cannot find the header.

How to fix

Rebase onto the latest main:

git fetch origin
git rebase origin/main
git push --force-with-lease

This will pick up the cJSON stub fix and the build should pass. No changes to your PR code are needed.

Other check results

Check Status
check-paths Passed
build Failed (cJSON header — CI infra, not your code)
analyze (cpp) Failed (same build error, so CodeQL could not run)
CodeQL Neutral (SARIF processing — unrelated)
check-windows Skipped

Automated analysis by AetherClaude

@ten9876
Copy link
Copy Markdown
Owner

ten9876 commented Apr 14, 2026

Taking via rebased branch — duplicate audio fixes from #1264 dropped, IQ stream support retained.

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.

TCI Android APP

2 participants