Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-11905

Fix PhotoCalib defintion to use multiplication

    Details

    • Story Points:
      4
    • Sprint:
      Alert Production F17 - 9, Alert Production F17 - 10
    • Team:
      Alert Production

      Description

      After further work on jointcal, I've realized that the basic definition of PhotoCalib should be multiplicative-instFlux * zeroPoint(x,y, instead of the current division-instFlux * zeroPoint(x,y. Fortunately, PhotoCalib isn't really used in the stack yet, so now is the time to fix it. Minor changes will also be required in jointcal and meas_mosaic creation of a PhotoCalib for persistence.

      This shouldn't have any impact on the existing API, just on the internal calculations and persistence.

      Not RFCing this, as it was defined this way in the original RFC and we changed it during implementation, and all of the current stakeholders have agreed to the change (slack#dm).

        Attachments

          Issue Links

            Activity

            Parejkoj John Parejko created issue -
            Parejkoj John Parejko made changes -
            Field Original Value New Value
            Link This issue relates to RFC-289 [ RFC-289 ]
            Parejkoj John Parejko made changes -
            Assignee John Parejko [ parejkoj ]
            Status To Do [ 10001 ] In Progress [ 3 ]
            Parejkoj John Parejko made changes -
            Link This issue blocks DM-9195 [ DM-9195 ]
            Parejkoj John Parejko made changes -
            Reviewers Jim Bosch, John Swinbank [ jbosch, swinbank ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            Parejkoj John Parejko made changes -
            Reviewers Jim Bosch, John Swinbank [ jbosch, swinbank ] John Swinbank, Lauren MacArthur [ swinbank, lauren ]
            Parejkoj John Parejko made changes -
            Summary Fix PhotoCalib defintion to use multiplciation Fix PhotoCalib defintion to use multiplication
            Parejkoj John Parejko made changes -
            Link This issue is triggering DM-12017 [ DM-12017 ]
            lauren Lauren MacArthur made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            swinbank John Swinbank made changes -
            Sprint Alert Production F17 - 9 [ 639 ] Alert Production F17 - 9, Alert Production F17 - 10 [ 639, 643 ]
            swinbank John Swinbank made changes -
            Rank Ranked higher
            Parejkoj John Parejko made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            swinbank John Swinbank made changes -
            Epic Link DM-11798 [ 34281 ]

              People

              • Assignee:
                Parejkoj John Parejko
                Reporter:
                Parejkoj John Parejko
                Reviewers:
                John Swinbank, Lauren MacArthur
                Watchers:
                Jim Bosch, John Parejko, John Swinbank, Lauren MacArthur, Paul Price, Robert Lupton, Russell Owen, Simon Krughoff
              • Votes:
                0 Vote for this issue
                Watchers:
                8 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel