-
Notifications
You must be signed in to change notification settings - Fork 28
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
L1 datamodel to ramp conversion #1198
Comments
Here's an example TVAC file that does not work with DQInitStep and the exception it produces. The file is at /grp/roman/TEST_DATA/RITA_DEV/TVAC_SAMPLES_RDM/TVAC1_NOMOPS_SCIMON_20231015055603_WFI01_uncal.asdf.
|
This issue is tracked on JIRA as RCAL-833. |
Comment by Jonathan Eisenhamer on JIRA: Tyler Desjardins/Javier Sanchez The given example data appear to need to be updated to rad/rdm version 0.20. The current data give the following when attempting to read:
Failed validating 'tag' in schema['properties']['meta']['allOf'][0]['allOf'][0]['properties']['calibration_software_version']: On instance['meta']['calibration_software_version']: |
Comment by Javier Sanchez on JIRA: Jonathan Eisenhamer , I just updated the file `███████████████████████████████████████████████████████████████████████████████████████████████████ to 0.20.0, and it validates correctly. Please let me know if it works for you. |
Comment by Jonathan Eisenhamer on JIRA: Much better. Thank you! |
Comment by Jonathan Eisenhamer on JIRA: Issue resolved by RCAL PR 1258 and RDM PR 352 |
Currently the logic to convert from a L1-like datamodel (e.g., WfiScienceRaw) to the Ramp model is contained in the DQInitStep:
This works for WfiScienceRaw, but not for other L1-like data such as TVAC or FPS datamodels. Either the code in DQInit needs to be updated to accept the other L1-like data (not including GW files), or a suggestion was to modify the datamodel to include a method that handles the transformation to the Ramp model.
The text was updated successfully, but these errors were encountered: