-
Notifications
You must be signed in to change notification settings - Fork 22
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
Split cal_step into L2 & L3 #363
Comments
This issue is tracked on JIRA as RAD-149. |
Comment by Nadia Dencheva on JIRA: Can you provide some more details as to what is the goal here? Split cal_step into two tags? Or do the split within the cal_step tag? Also what is the motivation? When split are the two tags (assuming two tags) going to be used in different ways? |
Maybe the better question for this ticket is if |
For L2 files, the cal_step resample never makes sense and will always take on some default value. The cal_steps outlier_detection and skymatch kind of make sense; they are applied to individual images and we should update them in their L2 image models when we do so. But we'll never set them in images we ordinarily send out. We should keep them in the L2 schema, though. For L3 files, I think the only cal_steps that could make sense are outlier_detection, skymatch, and resample. Resample must be complete if we have an L3 file, so it's a bit redundant but fine. skymatch and outlier_detection conceptually make sense though we'd populate them by looping over the input ImageModels and making sure that those have the skymatch and outlier detection steps complete. I think the options are:
I think my preference is for (2), but at rather low priority. |
Okay, let's go ahead and do (2), but only if we expect it to be a modest effort (<1 week). |
Split the cal_step schema into one for level 2 and one for level 3.
The text was updated successfully, but these errors were encountered: