[PIPE2D-1592] Implement S/N cut to FLUXSTD Created: 15/Nov/24  Updated: 22/Jan/25  Resolved: 22/Jan/25

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

Type: Task Priority: Normal
Reporter: Masayuki Tanaka Assignee: Masayuki Tanaka
Resolution: Done Votes: 0
Labels: EngRun, flux-calibration
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: PNG File snLambdaWindow.png    
Epic Link: flux calibration
Reviewers: price

 Description   

Some of the FLUXSTDs have very low S/N and they are useless. Let's implement a S/N cut for FLUXSTDs to use in fluxCal.



 Comments   
Comment by Masayuki Tanaka [ 18/Nov/24 ]

I propose to define the wavelength window to compute S/N in the r or m-arm because we reqire FLUXSTDs to have these arms for fluxCal. The exact wavelength window I suggest is 840-880nm. This is chosen as the region (1) without telluric absorption, (2) with average-ish sky lines, and (3) with CaII triplet lines, which are very important for stellar typing.

Comment by Masayuki Tanaka [ 18/Nov/24 ]

The spectra in the figure are pfsMerged.

Comment by Masayuki Tanaka [ 18/Dec/24 ]

As noted in PIPE2D-1518, this S/N cut is useful for identifying fibers that did not converge. Another reason to implement this feature. We want to implement the S/N cut not just in fitFluxCal.py but in fitPfsFluxReference.py as well so that we can exclude stars with no S/N entirely from fluxCal.

Comment by sogo.mineo [ 23/Dec/24 ]

What is the definition of "S/N"? Is it sum(flux) / sqrt(sum(variance))?

Comment by Masayuki Tanaka [ 24/Dec/24 ]

Sorry for being unclear. This is the median S/N per pixel in the 840-880nm wavelength window. So, if I write the per-pixel signal-to-noise as SN_i = flux_i / sqrt(variance_i), then the S/N I meant here is the median of SN_i within the wavelength window. I am still a bit worried about the sky residual, and I would prefer to adopt a robust S/N measurement.

Comment by sogo.mineo [ 20/Jan/25 ]

Could you review this pull request, too?

Comment by sogo.mineo [ 22/Jan/25 ]

Merged. Thanks for reviewing PR.

Generated at Tue Apr 08 12:02:24 JST 2025 using Jira 8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b.