[INSTRM-130] Refactor CCD header generation Created: 23/Jun/17 Updated: 28/May/20 Resolved: 28/May/20 |
|
| Status: | Done |
| Project: | Instrument control development |
| Component/s: | ics_ccdActor, ics_xcu_fpga |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Normal |
| Reporter: | cloomis | Assignee: | cloomis |
| Resolution: | Done | Votes: | 0 |
| Labels: | FITS, SPS | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||
| Story Points: | 7 | ||||||||||||||||||||
| Sprint: | SM1-2019 G, SM1-2019 K, SM1-2019 L, SM1-2019 N, SM1-2019 O, SM1-2019 P, SM1PD-2019 A, SM1PD-2019 B, SM1PD-2019 C, SM1PD-2019 D, SM1PD-2020 A, SM1PD-2020 B, SM1PD-2020 C, SM1PD-2020 D, SM1PD-2020 E, SM1PD-2020 F | ||||||||||||||||||||
| Description |
|
The ccdActor queries models to create the xcu, enu, dcb cards and concatenates the results into the FITS header. But the FPGA-level cards are returned as part of the Exposure, and the FPGA layer creates the HEADVERS card. That needs to change in response to actor changes (e.g. Make the FPGA headers come from a method other than readImage(), and have the ccdActor call it. |
| Comments |
| Comment by hassan [ 09/Jan/19 ] |
|
Partially done. Need to be completed. |
| Comment by hassan [ 18/Apr/19 ] |
|
FITS header changes potentially breaks other code. |
| Comment by hassan [ 23/May/19 ] |
|
Craig is contemplating going through keywords instead for everything. This may be a simpler solution. |
| Comment by hassan [ 19/Sep/19 ] |
|
Following ICS+DRP/SpS telecon 2019-09-18: considered lower priority, but to be implemented pre-shipping if time available, |
| Comment by cloomis [ 07/Dec/19 ] |
|
Merged the ics_actorkeys dictionary part at a790299 |
| Comment by cloomis [ 28/May/20 ] |
|
We grab some FPGA/FEE data at the start of integration and just before readout, and directly inject those into the ccd model. We need to add more of this, particularily for the readout voltages. Other tickets. Merged at 0d3b000, tagged 1.6.2 |