Repository navigation
Commit d267b41
Move camera delay to waveform constants (#1245)
* Move camera delay to waveform constants
* Fix camera delay migration and reload behavior
Goal:\nComplete the camera-delay move to waveform constants without regressing legacy configurations or leaving DAQ devices on stale timing after a waveform file reload.
Root cause:\nWaveform verification considered camera.delay but not the older camera.delay_percent field. DAQBase also retained the original waveform_constants mapping even though the reload path replaces the root mapping.
Changes:\n- fall back from camera.delay to camera.delay_percent when camera_delay is missing or invalid\n- expose the existing DAQ waveform_constants attribute as a live property\n- remove obsolete camera timing fields from shipped and reference configurations\n- document millisecond units, global scope, reload behavior, migration, and legacy calibration values\n- remove the unrelated AGENTS.md ignore entry and format PR-owned test changes
Regression coverage:\nAdd tests for legacy-field precedence, delay_percent migration, shipped synthetic configuration structure, and replaced waveform mappings in base and NI DAQ implementations.
Reuse analysis:\nReuse the existing waveform_constants attribute and all current DAQ callers; no new refresh API, public resource owner, or duplicate state lifecycle is introduced.
Verification:\n- 158 focused tests passed\n- Black 22.10.0 passed on files without known baseline debt\n- git diff --check passed\n- all 11 shipped/reference YAML files parsed with no camera delay fields\n- Sphinx dummy build parsed edited pages with zero page diagnostics\n- canonical Sphinx build remains blocked by unrelated missing gui_settings_popup autosummary modules
---------
Co-authored-by: Kevin M. Dean <kevin.dean@utsouthwestern.edu>1 parent c33333e commit d267b41
35 files changed
Lines changed: 246 additions & 85 deletions
File tree
- docs/source
- 02_user_guide
- 01_supported_hardware
- 03_user_interface_walkthrough
- 04_microscope_setup/01_software_configuration
- 05_reference/implementations
- configurations
- test
- config
- model
- devices
- daq
- galvo
- remote_focus
- stages
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
10 | 15 | | |
11 | 16 | | |
12 | 17 | | |
| |||
46 | 51 | | |
47 | 52 | | |
48 | 53 | | |
49 | | - | |
50 | 54 | | |
51 | 55 | | |
52 | 56 | | |
| |||
87 | 91 | | |
88 | 92 | | |
89 | 93 | | |
90 | | - | |
91 | 94 | | |
92 | 95 | | |
93 | 96 | | |
| |||
117 | 120 | | |
118 | 121 | | |
119 | 122 | | |
120 | | - | |
121 | 123 | | |
122 | 124 | | |
123 | 125 | | |
| |||
147 | 149 | | |
148 | 150 | | |
149 | 151 | | |
150 | | - | |
151 | 152 | | |
152 | 153 | | |
153 | 154 | | |
| |||
178 | 179 | | |
179 | 180 | | |
180 | 181 | | |
181 | | - | |
182 | 182 | | |
183 | 183 | | |
184 | 184 | | |
| |||
215 | 215 | | |
216 | 216 | | |
217 | 217 | | |
218 | | - | |
219 | 218 | | |
220 | 219 | | |
221 | 220 | | |
| 221 | + | |
222 | 222 | | |
223 | 223 | | |
224 | 224 | | |
| |||
241 | 241 | | |
242 | 242 | | |
243 | 243 | | |
244 | | - | |
245 | 244 | | |
246 | 245 | | |
247 | 246 | | |
| |||
264 | 263 | | |
265 | 264 | | |
266 | 265 | | |
| 266 | + | |
267 | 267 | | |
268 | 268 | | |
269 | 269 | | |
| |||
288 | 288 | | |
289 | 289 | | |
290 | 290 | | |
291 | | - | |
292 | 291 | | |
293 | 292 | | |
294 | 293 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
91 | 91 | | |
92 | 92 | | |
93 | 93 | | |
94 | | - | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
95 | 99 | | |
96 | 100 | | |
97 | 101 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
339 | 339 | | |
340 | 340 | | |
341 | 341 | | |
| 342 | + | |
| 343 | + | |
| 344 | + | |
| 345 | + | |
| 346 | + | |
| 347 | + | |
| 348 | + | |
| 349 | + | |
| 350 | + | |
| 351 | + | |
| 352 | + | |
| 353 | + | |
342 | 354 | | |
343 | 355 | | |
344 | 356 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
32 | 32 | | |
33 | 33 | | |
34 | 34 | | |
35 | | - | |
36 | 35 | | |
37 | 36 | | |
38 | 37 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
42 | 42 | | |
43 | 43 | | |
44 | 44 | | |
45 | | - | |
46 | 45 | | |
47 | 46 | | |
48 | 47 | | |
| |||
199 | 198 | | |
200 | 199 | | |
201 | 200 | | |
202 | | - | |
203 | 201 | | |
204 | 202 | | |
205 | 203 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
15 | 15 | | |
16 | 16 | | |
17 | 17 | | |
18 | | - | |
19 | 18 | | |
20 | 19 | | |
21 | 20 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
19 | 19 | | |
20 | 20 | | |
21 | 21 | | |
22 | | - | |
23 | 22 | | |
24 | 23 | | |
25 | 24 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
22 | 22 | | |
23 | 23 | | |
24 | 24 | | |
25 | | - | |
26 | 25 | | |
27 | 26 | | |
28 | 27 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
19 | 19 | | |
20 | 20 | | |
21 | 21 | | |
22 | | - | |
23 | 22 | | |
24 | 23 | | |
25 | 24 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
40 | 40 | | |
41 | 41 | | |
42 | 42 | | |
43 | | - | |
44 | 43 | | |
45 | 44 | | |
46 | 45 | | |
| |||
251 | 250 | | |
252 | 251 | | |
253 | 252 | | |
254 | | - | |
255 | 253 | | |
256 | 254 | | |
257 | 255 | | |
| |||
0 commit comments