[PIPE2D-676] Don't split arms/spectrographs in generateReductionSpec Created: 10/Dec/20 Updated: 22/Jan/21 Resolved: 22/Jan/21 |
|
| Status: | Done |
| Project: | DRP 2-D Pipeline |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Story | Priority: | Normal |
| Reporter: | price | Assignee: | sogo.mineo |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Sprint: | 2DDRP-2021 A | ||||||||
| Reviewers: | price | ||||||||
| Description |
|
generateReductionSpec.py currently generates separate YAML blocks for different arms. So long as the exposures from different arms are coordinated, we should group the blocks for different arms together (so that the YAML is shorter and simpler, and for efficiency in processing). |
| Comments |
| Comment by sogo.mineo [ 14/Jan/21 ] |
|
Do all commands from constructBias.py through reduceArc.py work gracefully if they are given a mixture of files of different arms? |
| Comment by price [ 15/Jan/21 ] |
|
Yes, they should. As an example, see the weekly. |
| Comment by sogo.mineo [ 18/Jan/21 ] |
|
I added a merge process. Could you review it? |
| Comment by sogo.mineo [ 22/Jan/21 ] |
|
Merged to master. Thanks for review. |