-
Notifications
You must be signed in to change notification settings - Fork 0
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
Karr/calib proto #53
Conversation
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
There was a problem hiding this 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.
Co-authored-by: Hugo Buddelmeijer <[email protected]>
Co-authored-by: Hugo Buddelmeijer <[email protected]>
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. |
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. |
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. |
It seemed you accidentally closed this instead of merging it, so I took the liberty to merge it anyway. |
Update which