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

Examine ref and source flux error propagation in PhotoCalTask and colorterms

    XMLWordPrintable

    Details

    • Type: Story
    • Status: To Do
    • Resolution: Unresolved
    • Fix Version/s: None
    • Component/s: pipe_tasks
    • Labels:
      None

      Description

      The code for the error propagation in PhotoCalTask and colorterms.py seems fishy to me, and is not really tested by the existing tests. We should work out the math for propagating these errors, implement tests of it, check that the code behaves correctly, and redo it if necessary.

      In particular, the interaction between Colorterm.propagateFluxErrors() and calib.absMagErrFromFluxErr() needs to be checked.

      We're not helped by the fact that the ref flux errors were originally totally ignored and/or faked (see DM-2308), and reference catalogs weren't guaranteed to even have flux errors (until RFC-535).

        Attachments

          Issue Links

          is triggered by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-13054 Add colorterm config support to jointcal

          • Done
          relates to

          Improvement - An improvement or enhancement to an existing feature or task. DM-2308 Improve hacks in PhotoCalTask for dealing with reference flux uncertainties

          • To Do

            Activity

            Ascending order - Click to sort in descending order
            Parejkoj John Parejko created issue -
            Parejkoj John Parejko made changes -
            Field Original Value New Value
            Link This issue is triggered by DM-13054 [ DM-13054 ]
            Parejkoj John Parejko made changes -
            Link This issue relates to DM-2308 [ DM-2308 ]
            swinbank John Swinbank made changes -
            Labels SciencePipelines
            swinbank John Swinbank made changes -
            Epic Link DM-16722 [ 235355 ]
            swinbank John Swinbank made changes -
            Team Alert Production [ 10300 ]
            swinbank John Swinbank made changes -
            Epic Link DM-16722 [ 235355 ] DM-17887 [ 240317 ]
            swinbank John Swinbank made changes -
            Epic Link DM-17887 [ 240317 ] DM-19979 [ 307530 ]
            swinbank John Swinbank made changes -
            Epic Link DM-19979 [ 307530 ] DM-21441 [ 423048 ]
            swinbank John Swinbank made changes -
            Epic Link DM-21441 [ 423048 ] DM-22484 [ 427311 ]
            swinbank John Swinbank made changes -
            Epic Link DM-22484 [ 427311 ] DM-24339 [ 433026 ]
            swinbank John Swinbank made changes -
            Epic Link DM-24339 [ 433026 ] DM-25139 [ 435257 ]
            swinbank John Swinbank made changes -
            Epic Link DM-25139 [ 435257 ] DM-26810 [ 439762 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-26810 [ 439762 ] DM-27906 [ 442554 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-27906 [ 442554 ] DM-29214 [ 459218 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-29214 [ 459218 ] DM-30436 [ 504824 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-30436 [ 504824 ] DM-30501 [ 510159 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-30501 [ 510159 ] DM-30502 [ 510160 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-30502 [ 510160 ] DM-30506 [ 510172 ]
            horvat Nikolina Horvat made changes -
            Epic Link DM-30506 [ 510172 ] DM-34931 [ 1598492 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-34931 [ 1598492 ] DM-36006 [ 1997396 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-36006 [ 1997396 ] DM-36522 [ 2254239 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-36522 [ 2254239 ] DM-36523 [ 2254240 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-36523 [ 2254240 ] DM-36524 [ 2254243 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-36524 [ 2254243 ] DM-36525 [ 2254245 ]

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              Parejkoj John Parejko
              Watchers:
              Eli Rykoff, Jim Bosch, John Parejko, John Swinbank, Meredith Rawls, Russell Owen, Simon Krughoff, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

                Dates

                Created:
                Updated:

                  Jenkins

                  No builds found.