From f81c4ee5310309ade349c6c17cedf17722aceb87 Mon Sep 17 00:00:00 2001 From: x01dc Date: Thu, 17 Sep 2026 12:30:13 +0200 Subject: [PATCH] docs(ids-cameras): mark manual exposure/auto-gain plan as done Confirmed merged into main (round-4 HW-testing commit 2f6f445 is an ancestor of main) -- update the plan's status from "pending real-hardware verification" to done. Co-Authored-By: Claude Sonnet 5 Claude-Session: https://claude.ai/code/session_01Lx3KffiFyyDMKT8vvPENUW --- docs/plans/ids-camera-manual-exposure.md | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/docs/plans/ids-camera-manual-exposure.md b/docs/plans/ids-camera-manual-exposure.md index ce299e54..a9ba4965 100644 --- a/docs/plans/ids-camera-manual-exposure.md +++ b/docs/plans/ids-camera-manual-exposure.md @@ -1,10 +1,11 @@ # Plan: Manual exposure / auto-gain control for IDSCamera + xrayeye widget knobs -Status: **implemented on `feat/ids-camera-manual-exposure`, pending real-hardware -verification** (`IDSCamera` and the `OMNY_XRayEye` GUI widget are both used in -production during beamtimes; this change should be reviewed and verified against -real hardware outside a live beamtime -- see `csaxs_bec/device_configs/test_ids_camera_41.yaml` -for a single-camera (ID 41, color) config for that purpose -- before merging). +Status: **done — implemented and merged into `main`**, after four rounds of +real-hardware verification (see the addenda below for the bugs found and fixed +each round: startup auto-exposure/gain never actually engaging, a zero-ms +exposure floor, auto-gain being more useful as a one-time startup pulse than a +persistent toggle, and the pixel-clock slider needing to be index-based over +the camera's real discrete list instead of a ranged value). ## Context