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

Write and test iterative solution

    Details

    • Story Points:
      6
    • Sprint:
      Alert Production S17 - 1, Alert Production S17 - 2, Alert Production S17 - 3
    • Team:
      Alert Production

      Description

      A possible refinement to the DCR algorithm would be to calculate the best fit solution using a single DCR plane for the entire band (much like a standard coadd), and then fit for a delta off of that solution for each sub-band. This ticket will write and test that approach, but only merge if it looks promising.

        Attachments

          Issue Links

            Activity

            Hide
            rowen Russell Owen added a comment -

            I was relying on the pull request. Can you please fix it to include the changes that are relevant?

            Show
            rowen Russell Owen added a comment - I was relying on the pull request. Can you please fix it to include the changes that are relevant?
            Hide
            rowen Russell Owen added a comment -

            Thank you for pointing out the files I missed (github was not showing diffs for the two most important files). I have now reviewed them on github.

            Show
            rowen Russell Owen added a comment - Thank you for pointing out the files I missed (github was not showing diffs for the two most important files). I have now reviewed them on github.
            Hide
            sullivan Ian Sullivan added a comment -

            If you'd like to take a second look at the DCR template generation code, I believe I've now addressed everything from your first review. I have expanded and hopefully clarified the documentation, and have broken the two gigantic files into several smaller modules. I'm also happy to leave things as they are and finally close this ticket, though.

            Show
            sullivan Ian Sullivan added a comment - If you'd like to take a second look at the DCR template generation code, I believe I've now addressed everything from your first review. I have expanded and hopefully clarified the documentation, and have broken the two gigantic files into several smaller modules. I'm also happy to leave things as they are and finally close this ticket, though.
            Hide
            rowen Russell Owen added a comment - - edited

            I'll have another look and try to focus on really important things.

            You have committed a bunch of files in doc that look auto-generated (including many in doc/_build). Did you really mean to do that?

            Show
            rowen Russell Owen added a comment - - edited I'll have another look and try to focus on really important things. You have committed a bunch of files in doc that look auto-generated (including many in doc/_build ). Did you really mean to do that?
            Hide
            rowen Russell Owen added a comment -

            Other than that it looks fine. I put a few small suggestions on github.

            Show
            rowen Russell Owen added a comment - Other than that it looks fine. I put a few small suggestions on github.

              People

              • Assignee:
                sullivan Ian Sullivan
                Reporter:
                sullivan Ian Sullivan
                Reviewers:
                Russell Owen
                Watchers:
                Ian Sullivan, Russell Owen
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel