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

make jointcal integration/validation test for lsstSim

    Details

    • Story Points:
      10
    • Sprint:
      Alert Production F16 - 10, Alert Production F16 - 11, Alert Production F16 - 11b, Alert Production F16 - 11c
    • Team:
      Alert Production

      Description

      We need an integration/validation test for jointcal on lsstSim data, to show that jointcal can run safely on lsstSim data processed through the stack.

        Attachments

          Issue Links

            Activity

            Hide
            sullivan Ian Sullivan added a comment -

            How carefully do you want me to go through the changes to processEimage.py? I assume it was automatically generated and I should skip it, but I'll look through it if you want.

            Show
            sullivan Ian Sullivan added a comment - How carefully do you want me to go through the changes to processEimage.py? I assume it was automatically generated and I should skip it, but I'll look through it if you want.
            Hide
            sullivan Ian Sullivan added a comment -

            Looks quite good overall. Feel free to ignore the comments about formatting, if you prefer.

            Show
            sullivan Ian Sullivan added a comment - Looks quite good overall. Feel free to ignore the comments about formatting, if you prefer.
            Hide
            Parejkoj John Parejko added a comment -

            Don't worry about processEImage: I just copied that over directly from the data repo. The only thing worth looking at in validation_data_jointcal is probably the compress* file. Everything else is just a new version of the butler repo.

            Show
            Parejkoj John Parejko added a comment - Don't worry about processEImage: I just copied that over directly from the data repo. The only thing worth looking at in validation_data_jointcal is probably the compress* file. Everything else is just a new version of the butler repo.
            Hide
            Parejkoj John Parejko added a comment -

            I believe I've answered all of your questions, one way or the other.

            Show
            Parejkoj John Parejko added a comment - I believe I've answered all of your questions, one way or the other.
            Hide
            Parejkoj John Parejko added a comment -

            I rebased the *-spacing changes onto a mostly-appropriate commit. Successfully re-ran lsstSim test on lsst-dev after commit changes and rebase.

            Merged both validation_data_jointcal and jointcal branches. Ticket done.

            Show
            Parejkoj John Parejko added a comment - I rebased the *-spacing changes onto a mostly-appropriate commit. Successfully re-ran lsstSim test on lsst-dev after commit changes and rebase. Merged both validation_data_jointcal and jointcal branches. Ticket done.

              People

              • Assignee:
                Parejkoj John Parejko
                Reporter:
                Parejkoj John Parejko
                Reviewers:
                Ian Sullivan
                Watchers:
                Ian Sullivan, John Parejko, Russell Owen
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel