[PIPE2D-385] Improve arm merging algorithm Created: 08/Mar/19  Updated: 28/Sep/19  Resolved: 12/Sep/19

Status: Done
Project: DRP 2-D Pipeline
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Task Priority: Normal
Reporter: hassan Assignee: price
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Relates
relates to PIPE2D-344 Implement arm merging Done
Story Points: 3
Sprint: 2DDRP-2019 C, 2DDRP-2019 H

 Description   

Following PIPE2D-344: improve the implemented merging algorithm, based on the old makePfsObject code.

price to discuss this with rhl.



 Comments   
Comment by price [ 08/Mar/19 ]

The merging algorithm itself is a weighted sum, and I've implemented that. However, I don't understand what the FluxTbl is for; I'd appreciate it if rhl could explain that.

Comment by hassan [ 05/Apr/19 ]

Quoting @rhl in a slack chat (#drp-2d) on 2019-03-11:

In PIPE2D-385 @paulprice wrote: "The merging algorithm itself is a weighted sum, and I've implemented that. However, I don't understand what the FluxTbl is for; I'd appreciate it if rhl could explain that."

It's in the datamodel, and there's a slightly hard-to-spot explanation (hint: HDU #2 == FLUXTBL) : "The wavelengths are specified via the WCS cards in the header (e.g. CRPIX1, CRVAL1) for HDU #1, and explicitly in the table for HDU #2. We chose these two representations for the data due to the difficulty in resampling marginally sampled data onto a regular grid, while recognising the convenience of such a grid when rebinning, performing PCAs, or stacking spectra. For highest precision the data in HDU #2 is likely to be used."

Comment by price [ 12/Sep/19 ]

This is a duplicate of DAMD-63.

Generated at Sat Feb 10 15:52:34 JST 2024 using Jira 8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b.