Skip to content

Conversation

@oczoske
Copy link
Collaborator

@oczoske oczoske commented Apr 17, 2025

This PR adds ExtraFitsKeywords to the various METIS subsystems. The goal is to automatise setting keywords consistent with current knowledge of the observing templates and METIS FITS keyword definitions (these definitions have gaps so expect updates to the settings at a later stage of the project).

  • METIS.yaml: headers/FITS_cfo_keywords.yaml
  • METIS_IMG_LM.yaml: headers/FITS_img_lm_keywords.yaml
  • METIS_IMG_N.yaml: headers/FITS_img_n_keywords.yaml
  • METIS_LMS.yaml: headers/FITS_lms_keywords.yaml
  • METIS_LSS.yaml: headers/FITS_lss_keywords.yaml
  • METIS_DET_IMG_LM.yaml: headers/FITS_det_img_lm_keywords.yaml
  • METIS_DET_IMG_N_Geosnap.yaml: headers/FITS_det_img_n_geosnap_keywords.yaml
  • METIS_DET_IFU.yaml: headers/FITS_det_ifu_keywords.yaml

Spectral keywords cannot be written yet due to a scopesim issue (AstarVienna/ScopeSim#644; conflicting redefinitions of __getitem__ in OpticsManager, Effect and SpectralTracelist).

@teutoburg
Copy link
Contributor

@oczoske
#-strings need !-prefix to be resolved

Wouldn't that be a suffix?

@teutoburg teutoburg moved this from 🆕 New to 🏗 In progress in ScopeSim-development Apr 17, 2025
@teutoburg teutoburg added instrument-specific Limited to a certain IRDB instrument package instrument definition Addition or modification of instrument YAMLs labels Apr 17, 2025
@oczoske
Copy link
Collaborator Author

oczoske commented Apr 21, 2025

Things that are problematic:

  • Spectral modes cannot access meta keywords (Conflicting definitions of __getitem__ in Effect and SpectralTraceList ScopeSim#644)
  • LSS mode cannot add filter to ESO.DPR.TECH: there is one yaml METIS_LSS.yaml covering all three filters.
  • INS.OPTI9 (mask/grism): The value should read GRISM_L etc. (according to the template manual). The best I can do for now is to set the trace file (TRACE_LSS_L.fits), but that should be translated somehow. One possibility might be to define grism_name in default.yaml and derive #spectral_traces.filename from that.

@oczoske oczoske marked this pull request as ready for review April 21, 2025 10:38
@oczoske oczoske requested review from eiseleb47 and teutoburg April 21, 2025 10:39
@oczoske
Copy link
Collaborator Author

oczoske commented Apr 21, 2025

I've listed everyone (but not their dogs) as reviewers. Please have a look at the FITS_*.yaml files whether you need additional keywords for your pipeline recipes.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need a way to select IFU mode as an LMS sub-mode, with DPR.TECH = IFU and IFU mechanism IN?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's in FITS_lms_keywords.yaml (OPTI4 for the LMS pickoff). I'm changing DPR.TECH to IFU, in accord with the templates.

@wkausch
Copy link

wkausch commented Apr 22, 2025

Hi Oliver, thx for the implementation of the keywords. For the LSS we'll need also the used slits (following the DRLD that's INS.OPTI3.NAME) and the corresponding grism for the N-band (INS.OPTI12.NAME). Also for the metis_lm_lss_wave recipe the keyword SEQ.WCU.LASERn.

@oczoske
Copy link
Collaborator Author

oczoske commented Apr 22, 2025

OPTI3 is in FITS_cfo_keywords.yaml. OPTI12 is now present, and along with OPTI9 gets the correct name. I've added the SEQ.WCU.LASERn but set them all to ON - in scopesim they'd be all on at the same time, and it's actually a bit tricky to decide whether they are on or off... Which lamp is used is currently in SEQ.LAMP.NAME (which is not an official keyword). Can you work with that?

@wkausch
Copy link

wkausch commented Apr 22, 2025

OPTI3 is in FITS_cfo_keywords.yaml. OPTI12 is now present, and along with OPTI9 gets the correct name. I've added the SEQ.WCU.LASERn but set them all to ON - in scopesim they'd be all on at the same time, and it's actually a bit tricky to decide whether they are on or off... Which lamp is used is currently in SEQ.LAMP.NAME (which is not an official keyword). Can you work with that?

Yes, for the time being that should be fine, many thanks!

Copy link
Collaborator

@hugobuddel hugobuddel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great! Thanks @oczoske !

All of them look fine. I didn't compare them with any documents though.

minimum_throughput: !!float 0.

- name : common_fits_keywords
decription : FITS keywords common to all modes
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops, thanks

Copy link
Contributor

@teutoburg teutoburg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know enough about this to give a meaningful review, but I don't see any immediate problems from the ScopeSim side of things.

@oczoske oczoske merged commit 160d77d into dev_master Apr 28, 2025
8 checks passed
@oczoske oczoske deleted the oc/subsystem_fits_keywords branch April 28, 2025 17:35
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in ScopeSim-development Apr 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

instrument definition Addition or modification of instrument YAMLs instrument-specific Limited to a certain IRDB instrument package METIS

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants