frank/ruvnet--RuView
1
0
Fork 0
mirror of https://github.com/ruvnet/RuView synced 2026-07-23 17:33:20 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(cli): keep proven mean-amplitude carrier for room features

Browse Source 
The max-variance-subcarrier carrier locked onto motion artifacts (not
breathing) and also had an out-of-bounds bug on variable CSI subcarrier
counts. Reverted to the mean-amplitude carrier, which is validated live to
detect breathing. Phase-based extraction on a stable subcarrier remains the
proper higher-SNR refinement (ADR-151 §4).

Co-Authored-By: claude-flow <ruv@ruv.net>
This commit is contained in:
ruv
2026-06-09 12:27:48 -04:00
parent dfd35b9c10
commit a194143819
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
v2/crates/wifi-densepose-cli/src/room.rs
+6 -2
View File
@@ -32,8 +32,12 @@ fn now_unix() -> i64 {
.unwrap_or(0)
}
/// One scalar per frame: mean amplitude across all subcarriers/streams.
/// Carries presence/motion energy and the breathing amplitude modulation.
/// Per-frame scalar: mean amplitude across all subcarriers/streams.
///
/// Carries presence/motion energy plus the breathing amplitude modulation.
/// (Validated live on the ESP32 — picks up breathing where a max-variance
/// subcarrier instead locks onto motion artifacts. A phase-based carrier on a
/// *stable* subcarrier is the proper higher-SNR refinement — ADR-151 §4.)
fn frame_scalar(frame: &CsiFrame) -> f32 {
let a = &frame.amplitude;
if a.is_empty() {