# Approve LDM-562 JointCal Requirements

XMLWordPrintable

## Details

• Type: RFC
• Status: Implemented
• Resolution: Done
• Component/s:
• Labels:
None

## Description

LDM-562 is now ready for review at https://docushare.lsst.org/docushare/dsweb/Get/Version-46745/LDM-562.pdf

This has been approved by Science Pipelines as part of DM-10732 and we now request formal approval by DMCCB.

Whilst LDM-562 is named "Level 2 System Requirements", the document and SysML model only includes the JointCal requirements.

## Activity

Hide
Robert Lupton added a comment -

Looks great, but there is one thing that I think we need to clarify.

Section 1.8 covers spatial variability of two types: fixed in time (filter variation) and per visit (cloud). Section 1.9 covers per-visit zero point variation. I think we need to add the statement to 1.8 that some of these spatially variable terms will have to be fit separately in each visit — exactly what we do in 1.8 and 1.9 can then be left up to the implementation.

So I'm happy to accept this RFC with the change to 1.8:

 Specification: JointCal shall include photometric model components that vary smoothly over either the full focal plane or individual devices and may be different for each visit. 

Show
Robert Lupton added a comment - Looks great, but there is one thing that I think we need to clarify. Section 1.8 covers spatial variability of two types: fixed in time (filter variation) and per visit (cloud). Section 1.9 covers per-visit zero point variation. I think we need to add the statement to 1.8 that some of these spatially variable terms will have to be fit separately in each visit — exactly what we do in 1.8 and 1.9 can then be left up to the implementation. So I'm happy to accept this RFC with the change to 1.8: Specification: JointCal shall include photometric model components that vary smoothly over either the full focal plane or individual devices and may be different for each visit.
Hide
Jim Bosch added a comment -

(Robert Lupton and I discussed this in person, so I'm okay with this change as well).

Show
Jim Bosch added a comment - ( Robert Lupton and I discussed this in person, so I'm okay with this change as well).
Hide
Tim Jenness added a comment -

I've made the change in the model and will upload the final version to DocuShare. Wil O'Mullane do you concur that we baseline this version?

Show
Tim Jenness added a comment - I've made the change in the model and will upload the final version to DocuShare. Wil O'Mullane do you concur that we baseline this version?
Hide
Wil O'Mullane added a comment -

yes lets get it baselined

Show
Wil O'Mullane added a comment - yes lets get it baselined
Hide
Tim Jenness added a comment -

Released to Docushare.

Show
Tim Jenness added a comment - Released to Docushare.

## People

• Assignee:
Tim Jenness
Reporter:
Tim Jenness
Watchers:
Donald Petravick, Gregory Dubois-Felsmann, Jim Bosch, John Parejko, John Swinbank, Kian-Tat Lim, Mario Juric, Robert Lupton, Tim Jenness, Wil O'Mullane, Xiuqin Wu [X] (Inactive)