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

Share ImageSelector from Twinkles

    XMLWordPrintable

    Details

    • Story Points:
      2
    • Sprint:
      Alert Production F17 - 7, Alert Production F17 - 8, Alert Production F17 - 9, Alert Production F17 - 10
    • Team:
      Alert Production

      Description

      Bring the good seeing ImageSelector from the Twinkles branch into the decam_hits repository. Add a goodSeeingCoadd config parameter to use it.

        Attachments

          Issue Links

            Activity

            Hide
            ebellm Eric Bellm added a comment -
            Show
            ebellm Eric Bellm added a comment - Code is here: https://github.com/lsst/obs_lsstSim/blob/twinkles_%23435/python/lsst/obs/lsstSim/selectImages.py (maybe also selectLsstImages.py)
            Hide
            krzys Krzysztof Findeisen added a comment -

            Review complete. I have suggestions at varying degrees of severity; in particular, I'd like to see the exception handling cleaned up before this gets merged.

            Show
            krzys Krzysztof Findeisen added a comment - Review complete. I have suggestions at varying degrees of severity; in particular, I'd like to see the exception handling cleaned up before this gets merged.
            Hide
            ebellm Eric Bellm added a comment -

            Krzysztof Findeisen Can you take another look?

            Show
            ebellm Eric Bellm added a comment - Krzysztof Findeisen Can you take another look?
            Hide
            krzys Krzysztof Findeisen added a comment -

            Looks good.

            Show
            krzys Krzysztof Findeisen added a comment - Looks good.

              People

              Assignee:
              ebellm Eric Bellm
              Reporter:
              krughoff Simon Krughoff
              Reviewers:
              Krzysztof Findeisen
              Watchers:
              Eric Bellm, Krzysztof Findeisen, Meredith Rawls, Simon Krughoff
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: