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

Audit and debug current zogy image differencing code

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: In Review
    • Resolution: Unresolved
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Story Points:
      10
    • Epic Link:
    • Sprint:
      AP S20-5 (April), AP S20-6 (May), AP F20-1 (June), AP F20-2 (July), AP F20-5 (October), AP F20-6 (November), AP S21-3 (February)
    • Team:
      Alert Production
    • Urgent?:
      No

      Description

      The current zogy implementation is assumed to have issues in:

      • Understand how the variance plane should propagate in zogy (convolution then whitening decorrelation).
      • Correct padding and centering of images and PSFs in Fourier space opeartions. (numerical accuracy in DFT vs. origin location and periodicity assumptions)
      • Correct calculation of the difference image variance plane. (variance of image k-space components vs. k-space components of variance plane)
      • zogy.py:384 padSize argument usage

        Attachments

          Issue Links

            Activity

            gkovacs Gabor Kovacs created issue -
            gkovacs Gabor Kovacs made changes -
            Field Original Value New Value
            Link This issue relates to DM-21868 [ DM-21868 ]
            gkovacs Gabor Kovacs made changes -
            Link This issue relates to DM-21920 [ DM-21920 ]
            gkovacs Gabor Kovacs made changes -
            Issue Type Story [ 10001 ] Bug [ 1 ]
            gkovacs Gabor Kovacs made changes -
            Description The current zogy implementation is assumed to have issues in:
             * Understand how the variance plane should propagate in zogy (convolution then whitening decorrelation).
             * Correct padding and centering of images and PSFs in Fourier space opeartions. (numerical accuracy in DFT vs. origin location and periodicity assumptions)

             * Correct calculation of the difference image variance plane. (variance of image k-space components vs. k-space components of variance plane)
            The current zogy implementation is assumed to have issues in:
             * Understand how the variance plane should propagate in zogy (convolution then whitening decorrelation).
             * Correct padding and centering of images and PSFs in Fourier space opeartions. (numerical accuracy in DFT vs. origin location and periodicity assumptions)

             * Correct calculation of the difference image variance plane. (variance of image k-space components vs. k-space components of variance plane)
             * zogy.py:384 padSize argument usage
            gkovacs Gabor Kovacs made changes -
            Description The current zogy implementation is assumed to have issues in:
             * Understand how the variance plane should propagate in zogy (convolution then whitening decorrelation).
             * Correct padding and centering of images and PSFs in Fourier space opeartions. (numerical accuracy in DFT vs. origin location and periodicity assumptions)

             * Correct calculation of the difference image variance plane. (variance of image k-space components vs. k-space components of variance plane)
             * zogy.py:384 padSize argument usage
            The current zogy implementation is assumed to have issues in:
             * Understand how the variance plane should propagate in zogy (convolution then whitening decorrelation).
             * Correct padding and centering of images and PSFs in Fourier space opeartions. (numerical accuracy in DFT vs. origin location and periodicity assumptions)

             * Correct calculation of the difference image variance plane. (variance of image k-space components vs. k-space components of variance plane)
             * {{zogy.py:384}} padSize argument usage
            swinbank John Swinbank made changes -
            Epic Link DM-22635 [ 427744 ]
            swinbank John Swinbank made changes -
            Team Alert Production [ 10300 ]
            swinbank John Swinbank made changes -
            Sprint AP S20-5 (April) [ 986 ]
            Story Points 10
            swinbank John Swinbank made changes -
            Link This issue relates to DM-12917 [ DM-12917 ]
            swinbank John Swinbank made changes -
            Assignee Gabor Kovacs [ gkovacs ]
            swinbank John Swinbank made changes -
            Epic Link DM-22635 [ 427744 ] DM-24342 [ 433029 ]
            gkovacs Gabor Kovacs made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            swinbank John Swinbank made changes -
            Sprint AP S20-5 (April) [ 986 ] AP S20-5 (April), AP S20-6 (May) [ 986, 987 ]
            gkovacs Gabor Kovacs made changes -
            Link This issue is parent task of DM-25115 [ DM-25115 ]
            gkovacs Gabor Kovacs made changes -
            Link This issue is parent task of DM-25174 [ DM-25174 ]
            swinbank John Swinbank made changes -
            Epic Link DM-24342 [ 433029 ] DM-25144 [ 435262 ]
            swinbank John Swinbank made changes -
            Sprint AP S20-5 (April), AP S20-6 (May) [ 986, 987 ] AP S20-5 (April), AP S20-6 (May), AP F20-1 (June) [ 986, 987, 1019 ]
            swinbank John Swinbank made changes -
            Sprint AP S20-5 (April), AP S20-6 (May), AP F20-1 (June) [ 986, 987, 1019 ] AP S20-5 (April), AP S20-6 (May), AP F20-1 (June), AP F20-2 (July) [ 986, 987, 1019, 1025 ]
            swinbank John Swinbank made changes -
            Sprint AP S20-5 (April), AP S20-6 (May), AP F20-1 (June), AP F20-2 (July) [ 986, 987, 1019, 1025 ] AP S20-5 (April), AP S20-6 (May), AP F20-1 (June), AP F20-2 (July), AP F20-3 (August) [ 986, 987, 1019, 1025, 1033 ]
            swinbank John Swinbank made changes -
            Sprint AP S20-5 (April), AP S20-6 (May), AP F20-1 (June), AP F20-2 (July), AP F20-3 (August) [ 986, 987, 1019, 1025, 1033 ] AP S20-5 (April), AP S20-6 (May), AP F20-1 (June), AP F20-2 (July), AP F20-4 (September) [ 986, 987, 1019, 1025, 1039 ]
            swinbank John Swinbank made changes -
            Rank Ranked lower
            swinbank John Swinbank made changes -
            Sprint AP S20-5 (April), AP S20-6 (May), AP F20-1 (June), AP F20-2 (July), AP F20-4 (September) [ 986, 987, 1019, 1025, 1039 ] AP S20-5 (April), AP S20-6 (May), AP F20-1 (June), AP F20-2 (July), AP F20-5 (October) [ 986, 987, 1019, 1025, 1046 ]
            swinbank John Swinbank made changes -
            Epic Link DM-25144 [ 435262 ] DM-26804 [ 439756 ]
            gkovacs Gabor Kovacs made changes -
            Link This issue is duplicated by DM-10513 [ DM-10513 ]
            gkovacs Gabor Kovacs made changes -
            Link This issue is duplicated by DM-11656 [ DM-11656 ]
            gkovacs Gabor Kovacs made changes -
            Link This issue is duplicated by DM-9441 [ DM-9441 ]
            sullivan Ian Sullivan made changes -
            Sprint AP S20-5 (April), AP S20-6 (May), AP F20-1 (June), AP F20-2 (July), AP F20-5 (October) [ 986, 987, 1019, 1025, 1046 ] AP S20-5 (April), AP S20-6 (May), AP F20-1 (June), AP F20-2 (July), AP F20-5 (October), AP F20-6 (November) [ 986, 987, 1019, 1025, 1046, 1052 ]
            sullivan Ian Sullivan made changes -
            Sprint AP S20-5 (April), AP S20-6 (May), AP F20-1 (June), AP F20-2 (July), AP F20-5 (October), AP F20-6 (November) [ 986, 987, 1019, 1025, 1046, 1052 ] AP S20-5 (April), AP S20-6 (May), AP F20-1 (June), AP F20-2 (July), AP F20-5 (October), AP F20-6 (November), AP S21-1 (December) [ 986, 987, 1019, 1025, 1046, 1052, 1059 ]
            sullivan Ian Sullivan made changes -
            Rank Ranked higher
            sullivan Ian Sullivan made changes -
            Epic Link DM-26804 [ 439756 ] DM-27910 [ 442603 ]
            sullivan Ian Sullivan made changes -
            Sprint AP S20-5 (April), AP S20-6 (May), AP F20-1 (June), AP F20-2 (July), AP F20-5 (October), AP F20-6 (November), AP S21-1 (December) [ 986, 987, 1019, 1025, 1046, 1052, 1059 ] AP S20-5 (April), AP S20-6 (May), AP F20-1 (June), AP F20-2 (July), AP F20-5 (October), AP F20-6 (November), AP S21-2 (January) [ 986, 987, 1019, 1025, 1046, 1052, 1064 ]
            sullivan Ian Sullivan made changes -
            Rank Ranked lower
            sullivan Ian Sullivan made changes -
            Sprint AP S20-5 (April), AP S20-6 (May), AP F20-1 (June), AP F20-2 (July), AP F20-5 (October), AP F20-6 (November), AP S21-2 (January) [ 986, 987, 1019, 1025, 1046, 1052, 1064 ] AP S20-5 (April), AP S20-6 (May), AP F20-1 (June), AP F20-2 (July), AP F20-5 (October), AP F20-6 (November), AP S21-3 (February) [ 986, 987, 1019, 1025, 1046, 1052, 1072 ]
            sullivan Ian Sullivan made changes -
            Rank Ranked higher
            sullivan Ian Sullivan made changes -
            Reviewers Ian Sullivan [ sullivan ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            Hide
            sullivan Ian Sullivan added a comment -

            Could you please add a comment describing the work this ticket covered, and in particular any answers to the issues raised in the description.

            Show
            sullivan Ian Sullivan added a comment - Could you please add a comment describing the work this ticket covered, and in particular any answers to the issues raised in the description.

              People

              Assignee:
              gkovacs Gabor Kovacs
              Reporter:
              gkovacs Gabor Kovacs
              Reviewers:
              Ian Sullivan
              Watchers:
              Gabor Kovacs, Ian Sullivan
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated: