-
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
RIX MvA : WCS calibrations in IMG-N data products #251
Comments
@astronomyk I don't understand, why would I know anything about this? |
This RIX should not exist because it is for the DRLS which is not under review. It will probably not reach the "Submitted" stage. Similar for the other DRLS related ones. If we do get them, then simply answer that the DRLS is not under review. But politely, because we don't want to make it sound like we wasted the reviewers time. |
Ah, I understand. This is not about the DRLS, but about the PIP Specification This is about https://polarion.astron.nl/polarion/#/project/METIS/workitem?id=METIS-6105 "IMG-N data products", which indeed does not list WCS calibration. The LM-band counterpart https://polarion.astron.nl/polarion/#/project/METIS/workitem?id=METIS-6104 "IMG-LM data products" does list as a last step
I don't know the procedure for updating the PIP specification, and I don't really care at the moment. So I will answer the question without mentioning we will update the PIP spec. Answered with
|
Ah, I also misread this. I can update the item on Polarion. Edit: ... and I answer Mario's comment in Jira. |
Thanks for doing the right thing @ivh and editing the PIP Spec. In the meantime we confirmed with @Rumpelstil that a) the omission of WCS calibration was deliberate, and b) that decision has been reverted and it should indeed be included. See also #263 I suppose we can now close this, as https://polarion.astron.nl/polarion/#/project/METIS/workitem?id=METIS-6105 has indeed been updated. Or maybe we should first change the status from Draft to Ready for Internal Review? |
Closing this since the AI is already carried out and closed on jira. |
https://jira.eso.org/browse/MET-2082
@ivh this is a throw back to the PIP Spec Doc. Would you like to comment?
The text was updated successfully, but these errors were encountered: