Skip to content

Add direct power trace foundation - #7

Draft
AlCalzone wants to merge 1 commit into
musithang:mainfrom
AlCalzone:alcalzone-power-trace-foundation
Draft

Add direct power trace foundation#7
AlCalzone wants to merge 1 commit into
musithang:mainfrom
AlCalzone:alcalzone-power-trace-foundation

Conversation

@AlCalzone

@AlCalzone AlCalzone commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

This PR adds the acquisition foundation for tinySA support. It builds on the merged geometry, display-capability, and shared-statistics prerequisites in #12, #13, and #14.

sdrtop's acquisition path only supported sample-streaming receivers. Spectrum analyzers that report calibrated power traces had no accurate path into its spectrum and waterfall views.

This PR adds direct power acquisition and compatible display layouts. #8 builds the tinySA backend on this foundation.

@musithang

Copy link
Copy Markdown
Owner

Thanks a lot! I'll review it tonight at home, or tomorrow morning at the latest, and then I'll merge it. :)

@AlCalzone

AlCalzone commented Sep 7, 2026

Copy link
Copy Markdown
Contributor Author

Three independent preparation drafts now precede the foundation: #12 aligns bin geometry, #13 moves display assumptions into device capabilities, and #14 shares acquisition behavior. After they land, #7 will be rebased onto main; the tinySA sequence then continues through #8, #9, #10, and #11. The corresponding dependent branches remain registered as native stack 4 in the fork because GitHub cannot register cross-fork PRs in an upstream native stack.

@musithang

Copy link
Copy Markdown
Owner

Perfect, thanks for structuring it so cleanly!

@AlCalzone

Copy link
Copy Markdown
Contributor Author

I'll set it ready for review when I'm done reviewing myself 😅

@musithang

Copy link
Copy Markdown
Owner

Sounds good, take your time! Let me know when it's ready. 👍

@AlCalzone

Copy link
Copy Markdown
Contributor Author

@musithang sorry for the PR churn. I didn't like how big this PR had gotten after a few local review rounds, so I split out a few things.

#12
#13
#14

are ready for your review now. We'll continue with this one after those are done.

@AlCalzone
AlCalzone force-pushed the alcalzone-power-trace-foundation branch from 4b2c404 to b6182eb Compare September 8, 2026 09:19
@musithang

musithang commented Sep 8, 2026

Copy link
Copy Markdown
Owner

#12, #13, #14 are all merged now. This one's showing conflicts against main since it still carries the same prep commits under different hashes. Go ahead and rebase whenever you're ready, that should drop the duplicates and leave just the actual tinySA foundation. I'll review and test locally again once it's up to date. Let me know when you ready.

Add backend-neutral calibrated trace acquisition, measured-point publication, power RX control, and capability-aware layout fallback. Preserve IQ behavior through the shared prerequisite contracts.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@AlCalzone
AlCalzone force-pushed the alcalzone-power-trace-foundation branch from b6182eb to 4189300 Compare September 8, 2026 18:27
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