[PIPE2D-851] reduceExposure fiber check doesn't respect spectrographs Created: 09/Jun/21 Updated: 15/Jun/21 Resolved: 15/Jun/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: | price |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Story Points: | 1 |
| Sprint: | 2DDRP-2021 A 6 |
| Reviewers: | Romain Philippe Lhoussaine ben brahim |
| Description |
|
When processing some b3 data, we get: RuntimeError: detectorMap (/data/drp/sps/CALIB/DETECTORMAP/pfsDetectorMap-024841-b3.fits) does not include fibers: [2, 63, 192, 255, 339, 401, 464, 525, 587, 650, 653, 714, 843, 906, 990, 1052, 1115, 1176, 1238, 1301, 1955, 2016, 2145, 2208, 2292, 2354, 2417, 2478, 2540, 2603] The detectorMap contains only the fibers applicable to that spectrograph module, but the pfsConfig includes all fibers for all spectrographs, and so there's a mismatch. |
| Comments |
| Comment by price [ 09/Jun/21 ] |
|
Romain Philippe Lhoussaine ben brahim, I’ve pushed a fix in tickets/ |
| Comment by hassan [ 09/Jun/21 ] |
|
Changes in pull request look fine to me. No comments. |
| Comment by price [ 15/Jun/21 ] |
|
Merged. |