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

Inconsistent application of Astier's amatrix in brighter fatter correction

    XMLWordPrintable

Details

    • Bug
    • Status: Done
    • Resolution: Done
    • None
    • None
    • None
    • 1
    • Data Release Production
    • No

    Description

      The current implementation of the Brighter Fatter correction takes amatrix from the full covariance fit as-is (they are all positive). However, because of inconsistency in the sign, it doesn't match the current implementation of Coulton's BF correction algorithm (it assumes negative). When it applies the correction, the sign of amatrix needs to be flipped by a -> -a.

      I don't know where this fix should go though.

      Attachments

        Activity

          youtsumi Yousuke Utsumi created issue -
          czw Christopher Waters made changes -
          Field Original Value New Value
          Epic Link DM-32177 [ 779901 ]
          Team Data Release Production [ 10301 ]
          abrought Alex Broughton made changes -
          czw Christopher Waters made changes -
          Status To Do [ 10001 ] In Progress [ 3 ]
          czw Christopher Waters made changes -
          Assignee Christopher Waters [ cwaters ]
          czw Christopher Waters made changes -
          Story Points 1
          czw Christopher Waters made changes -
          Reviewers Andrés Alejandro Plazas Malagón [ plazas ]
          Status In Progress [ 3 ] In Review [ 10004 ]
          plazas Andrés Alejandro Plazas Malagón made changes -
          Status In Review [ 10004 ] Reviewed [ 10101 ]
          czw Christopher Waters made changes -
          Resolution Done [ 10000 ]
          Status Reviewed [ 10101 ] Done [ 10002 ]

          People

            czw Christopher Waters
            youtsumi Yousuke Utsumi
            Andrés Alejandro Plazas Malagón
            Alex Broughton, Andrés Alejandro Plazas Malagón, Christopher Waters, Eli Rykoff, Yousuke Utsumi
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Jenkins

                No builds found.