From 00fef4a99d1ed8c547300ca28b25fbc542e1ad0b Mon Sep 17 00:00:00 2001 From: jjangddu Date: Mon, 28 Sep 2026 10:40:47 +0900 Subject: [PATCH] =?UTF-8?q?fix(hospital):=20ImuSidecar.restore=20=EA=B0=80?= =?UTF-8?q?=20=EB=82=A8=EC=9D=98=20=EC=BD=9C=EB=B0=B1=EC=9D=84=20=EB=90=98?= =?UTF-8?q?=EB=8F=8C=EB=A6=AC=EC=A7=80=20=EC=95=8A=EA=B2=8C=20=ED=95=9C?= =?UTF-8?q?=EB=8B=A4=20=C2=B7=20=EB=8B=A8=EA=B3=84=EB=B3=84=20=EB=A1=9C?= =?UTF-8?q?=EA=B7=B8?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## 계기 — 그리고 정정 2026-09-28 A34 에서 0cm 정렬 뒤 `align_0cm_imu.csv` 가 없다고 판단해 원인을 쫓았다. BLE 로그·parseRim 로그로 IMU 가 20/20 도착한 것까지 확인한 뒤 "사이드카 콜백이 덮였다"는 가설로 진단 로그를 넣어 재설치했다. 그런데 파일은 **처음부터 있었다** — 0cm·1cm 모두 09:49/09:51 에 20 cycle × 15 sample 로 써졌다. `adb shell` 이 보는 `/sdcard`(FUSE · MediaProvider 뷰)가 앱이 방금 쓴 파일을 한동안 안 보여 준 것이고, `find -type f` 는 한 파일만, `find -mmin` 은 아무것도 돌려주지 않았다. 진단은 불필요했다. 코드에 결함은 없었다. ## 그래도 남기는 것 - `restore()` 방어: 지금 걸린 콜백이 **자기 것일 때만** 되돌린다. LaunchedEffect 가 재시작하면 옛 인스턴스의 restore 가 새 인스턴스의 install 뒤에 실행될 수 있고 (취소된 코루틴의 finally 는 나중에 돈다 · 새 코루틴은 Main.immediate 로 즉시 시작), 그러면 새 훅이 옛 previous 로 덮여 그 자리 20회가 전부 IMU 없이 저장된다. 이번엔 그 순서가 아니었지만 순서 자체는 가능하다. - 로그(태그 ImuSidecar · AnchorAlign): install 시 이전 콜백 종류 · 콜백 도착 · await 결과 · 회차별 imu 수. "IMU 가 왔나" 는 파형 해석에서 제일 먼저 묻는 질문인데 답이 BLE 로그를 뒤져야 나왔다. 이제 logcat 한 줄로 답이 나온다. ## 교훈 (도구) 앱이 공용 Downloads 에 방금 쓴 파일은 `adb shell ls/find` 에 늦게 나타난다. 파일 유무를 근거로 결론 내리기 전에 `run-as`(앱 내부 경로) 로 보거나, 몇 분 뒤 다시 보거나, 앱이 남긴 로그·업로드 표식으로 교차 확인할 것. Co-Authored-By: Claude Fable 5.1 --- .../ui/views/clinical/AnchorAlignView.kt | 4 +- .../vesiscan/ui/views/clinical/ImuSidecar.kt | 47 +++++++++++++++++-- 2 files changed, 45 insertions(+), 6 deletions(-) diff --git a/app/src/main/java/com/medithings/vesiscan/ui/views/clinical/AnchorAlignView.kt b/app/src/main/java/com/medithings/vesiscan/ui/views/clinical/AnchorAlignView.kt index b84fb26..da8a9e7 100644 --- a/app/src/main/java/com/medithings/vesiscan/ui/views/clinical/AnchorAlignView.kt +++ b/app/src/main/java/com/medithings/vesiscan/ui/views/clinical/AnchorAlignView.kt @@ -326,7 +326,9 @@ fun AnchorAlignView(appState: AppState) { return@LaunchedEffect } // IMU 는 piezo 뒤에 온다. 없으면 빈 목록 — 측정을 실패로 돌리지 않는다. - cyclesImu.add(imuSide.await() ?: emptyList()) + val imu = imuSide.await() ?: emptyList() + cyclesImu.add(imu) + android.util.Log.d("AnchorAlign", "cycle $i/${AnchorConfig.ALIGN_CYCLES} imu=${imu.size}") val sorted = got.sortedBy { it.channel } liveChannels = sorted // 화면 파형 — 저장·판정에 쓰는 것과 같은 데이터 detachWatcher.push(sorted.map { it.buffer })?.let { d -> detachSnapshot = d } diff --git a/app/src/main/java/com/medithings/vesiscan/ui/views/clinical/ImuSidecar.kt b/app/src/main/java/com/medithings/vesiscan/ui/views/clinical/ImuSidecar.kt index a7484a8..6ee6ce5 100644 --- a/app/src/main/java/com/medithings/vesiscan/ui/views/clinical/ImuSidecar.kt +++ b/app/src/main/java/com/medithings/vesiscan/ui/views/clinical/ImuSidecar.kt @@ -18,9 +18,16 @@ * IMU 가 안 와도 측정은 성립한다 — 펌웨어·설정에 따라 `rim:` 이 없을 수 있고, 그때 * 측정을 실패로 돌리면 기존에 되던 일이 안 되게 된다. [await] 는 null 을 돌려주고 * 호출부는 IMU 없이 저장한다. + * + * ## 2026-09-28 진단 로그 · 방어적 restore + * A34 실기에서 0cm 정렬 20회가 `rim:` 을 전부 받았는데(BLE 로그 · parseRim 로그 확인) + * `align_0cm_imu.csv` 가 안 생겼다. 콜백 사슬 어디서 끊기는지 보려고 단계마다 로그를 + * 남긴다(태그 `ImuSidecar`). 아울러 [restore] 는 **지금 걸린 콜백이 자기 것일 때만** + * 되돌린다 — 옛 인스턴스의 restore 가 새 인스턴스의 install 을 덮는 순서가 가능하다. */ package com.medithings.vesiscan.ui.views.clinical +import android.util.Log import com.medithings.vesiscan.ble.BleManager import com.medithings.vesiscan.ble.ImuSample import kotlinx.coroutines.channels.Channel @@ -34,29 +41,59 @@ class ImuSidecar(private val ble: BleManager) { private val inbox = Channel>(Channel.CONFLATED) private var previous: ((List) -> Unit)? = null private var installed = false + private val id = Integer.toHexString(System.identityHashCode(this)) + + /** 우리가 건 콜백. restore 때 "아직 우리 것인가"를 참조로 비교한다. */ + private val mine: (List) -> Unit = { samples -> + val r = inbox.trySend(samples) + Log.d(TAG, "[$id] onComplete <- ${samples.size} samples · trySend ok=${r.isSuccess} closed=${r.isClosed}") + } /** 콜백을 가로챈다. 원래 콜백은 [restore] 에서 되돌린다. */ fun install() { if (installed) return previous = ble.imuCollector.onComplete - ble.imuCollector.onComplete = { samples -> inbox.trySend(samples) } + ble.imuCollector.onComplete = mine installed = true + Log.d( + TAG, + "[$id] install · previous=${previous?.javaClass?.name ?: "null"}" + + " · ble=${Integer.toHexString(System.identityHashCode(ble))}" + + " · collector=${Integer.toHexString(System.identityHashCode(ble.imuCollector))}", + ) } /** 직전 회차에서 늦게 도착한 것을 버린다. piezo 쪽 비우기와 같은 자리에서 부른다. */ fun drain() { - while (inbox.tryReceive().isSuccess) { /* 비우기 */ } + var n = 0 + while (inbox.tryReceive().isSuccess) n++ + if (n > 0) Log.d(TAG, "[$id] drain · dropped=$n") } /** piezo 를 받은 뒤 부른다. 시간 안에 안 오면 null — 호출부는 IMU 없이 진행한다. */ - suspend fun await(graceMs: Long = IMU_GRACE_MS): List? = - withTimeoutOrNull(graceMs) { inbox.receive() } + suspend fun await(graceMs: Long = IMU_GRACE_MS): List? { + val hookIsMine = ble.imuCollector.onComplete === mine + val got = withTimeoutOrNull(graceMs) { inbox.receive() } + Log.d(TAG, "[$id] await -> ${got?.size?.toString() ?: "TIMEOUT"} · hookIsMine=$hookIsMine") + return got + } fun restore() { if (!installed) return - ble.imuCollector.onComplete = previous + val current = ble.imuCollector.onComplete + if (current === mine) { + ble.imuCollector.onComplete = previous + Log.d(TAG, "[$id] restore · ok") + } else { + // 우리 뒤에 누군가 다시 걸었다 — 그 사람 것을 되돌리면 안 된다. + Log.w(TAG, "[$id] restore · SKIPPED — hook is not mine (${current?.javaClass?.name ?: "null"})") + } previous = null installed = false inbox.close() } + + private companion object { + const val TAG = "ImuSidecar" + } }