[INSTRM-1149] Add basic data QA checks at acquisition Created: 24/Dec/20 Updated: 24/Dec/21 Resolved: 24/Dec/21 |
|
| Status: | Done |
| Project: | Instrument control development |
| Component/s: | ics_ccdActor |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Normal |
| Reporter: | cloomis | Assignee: | arnaud.lefur |
| Resolution: | Done | Votes: | 0 |
| Labels: | SPS | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Story Points: | 2 |
| Sprint: | SM1PD-2020 N, SM1PD-2021 A, SM1PD-2021 A 2, SM1PD-2021 A 3, SM1PD-2021 A10, SM1PD-2021 A 11, SM1PD-2021 A 12, SM1PD-2021 A 13, SM1PD-2021 A 15, SM1PD-2021 A16 |
| Reviewers: | cloomis |
| Description |
|
The ccdActor has the actual image in RAM before writing it to disk, so should perform reasonable QA checks right then. In particular, it should check overscan level and noise for each amp. This would have caught the misconfiguration from the 2020-12-17 work sooner than 2020-12-23... Reported on some keywords. I think we'd like this to make it out to STS, so I propose at least: visitQA=$visit,status, where status is the usual "OK" vs. "something informative". Do we also want overscanLevels=8*float and overscanNoise=8*float? No harm in it I don't think, but gets into choices: smallish rectangle, remove drift/tilt, or live with stats on whole thing? Etc. Note that this touches on the warning-vs-error-vs-critical topic. |
| Comments |
| Comment by arnaud.lefur [ 24/Dec/21 ] |
|
merged as 1.8.1 |