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

Karr/calib proto #53

Merged
merged 6 commits into from
Oct 24, 2024
Merged

Karr/calib proto #53

merged 6 commits into from
Oct 24, 2024

Conversation

JenniferKarr
Copy link
Contributor

Update which

  • generates prototypes for the external calibration files
  • contains a minimal set of SOF files for all recipes
  • has a few minor bug fixes and updates

files. PRO.CATG is set according to DRLD values.

Added a command line option --doStatic to run_recipes to trigger the code
  - templates in sofTemplates/
  - versions to use in sofFiles/
Added prototypes for external calibrations; generated as part of script.

A few minor fixes
Copy link
Contributor

@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! Now we have all the pieces of the puzzle, thanks!

I didn't go through everything in makeCalibPrototypes.py, so I'll trust you that it is okay, and we can fix anything later.

I was not aware one can use asterisks in SOF files, but maybe we can. Otherwise we will know soon enough :-).

Edit: please note that I did add some small suggestions.

ESO/grabHeaders.py Outdated Show resolved Hide resolved
ESO/grabHeaders.py Outdated Show resolved Hide resolved
JenniferKarr and others added 2 commits October 24, 2024 10:03
Co-authored-by: Hugo Buddelmeijer <[email protected]>
Co-authored-by: Hugo Buddelmeijer <[email protected]>
@JenniferKarr
Copy link
Contributor Author

Regarding the asterisks, those are a hack for generating sof files without having to redo them manually if the file names change (and without recreating the EDPS header parsing system). The script reads in the templates, grabs the relevant RAW files by DO.CATG if there is an asterisk in the file name, and writes the output files; after that there's about five minutes of tidying.

@JenniferKarr
Copy link
Contributor Author

Added SOF files and prototypes for the external calibration files. SOF files in sofFiles/ match the simulated images in output/ and have the filename of the recipes.

Note that the actual tags in the SOF files can be anything (as set by the workflow); I've used the DO.CATG/PRO.CATG as given in the DRLD as the defaults. It's probably simpler to stick with these until the automatic generation of SOF files is working.

@hugobuddel
Copy link
Contributor

Yeah the idea behind the DO.CATG values is to use them as tags in the SOF files for the recipes. In fact, the DO.CATG is a transient 'thing' used exactly for that purpose by the OCA rules, a predecessor of the EDPS. So we should use those values in the recipes too, that is why they are in the DRLD.

@hugobuddel hugobuddel reopened this Oct 24, 2024
@hugobuddel hugobuddel merged commit bbe64a9 into main Oct 24, 2024
2 checks passed
@hugobuddel
Copy link
Contributor

It seemed you accidentally closed this instead of merging it, so I took the liberty to merge it anyway.

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

Successfully merging this pull request may close these issues.

2 participants