Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RIX PWb : Reduced 2D format #249

Open
astronomyk opened this issue Oct 23, 2023 · 3 comments
Open

RIX PWb : Reduced 2D format #249

astronomyk opened this issue Oct 23, 2023 · 3 comments
Assignees
Labels
Milestone

Comments

@astronomyk
Copy link
Contributor

https://jira.eso.org/browse/MET-2079

It is a very good idea to store the reduced 2D detector images in such a way that they can be easily combined with other exposures later. However, it is not clear to me what data structure is really foreseen to do that. Saving the full wavelength solution plus slice location/distortion in a FITS header seems heavy. A table extension can carry more data efficiently, but also does not seem to be optimal for this case. Perhaps this should be described in Sect. 8.2.3.1 (p.241) on the IFU_SCI_REDUCED format.

@hugobuddel , pass this on if e.g. @sesquideus could tackle this

@hugobuddel
Copy link
Contributor

I have no clue. We state in 6.6.3 metis_ifu_reduce

The recipe produces the following (intermediate) data products:
• Reduced 2D detector images. These are accompanied by additional information describing the geometry of the slice layout, target position and wavelength calibration to the ex tent that the exposure can be combined with other exposures into a single rectified spectral cube. This information can be stored in the FITS header or a table extension.
...

But then in section 8.2.3.1 IFU_SCI_REDUCED we don't explain how this is done at all.

I don't really understand how the IFU_SCI_REDUCED is supposed to be combined. 8.2.3.1 says that IFU_SCI_REDUCED (a 2D image) is input to metis_ifu_calibrate which produces a IFU_SCI_CUBE_CALIBRATED (a 3D cube). I believe this is an error.

I propose something like

The precise data structure for the auxiliary information for IFU_SCI_REDUCED (reduced 2D detector images) is not yet properly defined. We will define this data structure and describe it in 8.2.3.1 (IFU_SCI_REDUCED).

OK @sesquideus ?

@hugobuddel
Copy link
Contributor

Answered with

The precise data structure for the auxiliary information for IFU_SCI_REDUCED (reduced 2D detector images) is not yet fully defined. We will define this data structure and describe it in 8.2.3.1 (IFU_SCI_REDUCED).

Still quite some work to actually do that

@sesquideus
Copy link
Collaborator

I'm fine with this. Sadly I have little idea about the data structures yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants