[PIPE2D-347] Provide a method that provides a PSF for a given fiber and wavelength Created: 08/Feb/19  Updated: 10/Apr/20  Resolved: 07/Dec/19

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

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

Issue Links:
Blocks
blocks PIPE2D-326 Implement arc subtraction algorithm i... Done
is blocked by PIPE2D-353 Provide a goodness-of-fit for PSF and... Won't Fix
Relates
relates to PIPE2D-547 Investigate and solve the problem of ... Done
relates to PIPE2D-450 Create array_of_arrays that contains ... Done
relates to PIPE2D-479 Center of light difference between 10... Done
Story Points: 8
Epic Link: 2D PSF modelling
Sprint: 2DDRP-2019 J, 2DDRP-2019 K

 Description   

Provided a method that, given a fiber and wavelength, provides the appropriate Psf instance.

This method may need to support multiple implementations.



 Comments   
Comment by ncaplar [ 04/Oct/19 ]

hassan Is it ok to hijack this ticket for my function which provides 2D PSF? (following the discussion and comment by price: `Regarding the 2D PSF, I suggest that Neven provides me a function that generates his PSF, and I'll wrap it up in the API. How that gets ticketed doesn't bother me.`) Note that I would probably not provide PSF as function of wavelength as it says in the description, but x and y (which is what I believe 2D PSF class will also take).

Comment by hassan [ 04/Oct/19 ]

Yes, no problem using this ticket for your function ncaplar.

Comment by ncaplar [ 16/Oct/19 ]

@price Paul, for the lack fo clear location, I have placed the first version of the code at https://github.com/nevencaplar/PFS_Work_In_Progress/blob/master/PIPE2D-347/Provide_PSF_2D.py

As you might see, it depends on the files in Tiger, which are placed at /tigress/ncaplar/PIPE2D-450.

Very happy to help with clarification or code! Excited to see this first version implemented so that we can start working on improvements!

 

Comment by price [ 02/Nov/19 ]

I've got a basic implementation going; work is on PIPE2D-343.

ncaplar, can you regenerate your images with the PSF centered on the middle pixel? Having it centered between two pixels creates an offset when binning by 5.

Comment by ncaplar [ 02/Nov/19 ]

I do not fully understand the request/problem. You basically would want to an odd number of pixels in the provided PSF image? Can you problem be solved by shaving off last few columns/rows of the provided image? Would kind of offset is being created when binning by 5?

 

Comment by price [ 02/Nov/19 ]

It doesn't matter how many pixels there are — I can deal with the edges.

I want the centroid of the image to be centered on a pixel after binning by 5. This cannot be done by changing the origin when binning, because 5 is an odd number. Currently, the centroid is halfway between two pixels. No matter where I place a 5x5 pixel box around that point, the centroid cannot be in the center of that box.

Actually, maybe it would be best if I measure the centroid and interpolate so that it's centered? That way, I can remove any coordinate differences between your code and the pipeline.

Comment by rhl [ 03/Nov/19 ]

Let's not re-estimate the centroid – therein lie biases and offsets. Neven, can you generate the PSF centred in a pixel (which is 0.0, 0.0 in the coordinates we use?)

Comment by ncaplar [ 04/Nov/19 ]

I still do not understand the problem, and I think some more explaining will be needed on Monday. Having said that, I do not expect any problems here, I can generate either 10x oversampled PSFs or somehow shifted PSF (once when I understand what the problem is).

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