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

Add minimum and maximum lambda to filter properties

    Details

    • Story Points:
      3
    • Sprint:
      AP S18-4, AP S18-5
    • Team:
      Alert Production

      Description

      The DCR correction code needs to know the range of wavelengths permitted through a filter, not just the effective wavelength of the full band. I will add additional properties to `Filter` and define those properties for `obs_lsstSim` and `obs_decam`. Other packages should continue working as normal without being modified.

        Attachments

          Issue Links

            Activity

            Hide
            sullivan Ian Sullivan added a comment -

            I've added a simple unit test of the FilterProperty additions.

            Show
            sullivan Ian Sullivan added a comment - I've added a simple unit test of the FilterProperty additions.
            Hide
            jmeyers314 Joshua Meyers added a comment -

            Looks good to me.

            I left a couple of questions for you on GitHub.

            Show
            jmeyers314 Joshua Meyers added a comment - Looks good to me. I left a couple of questions for you on GitHub.
            Hide
            sullivan Ian Sullivan added a comment -

            Thank you for the quick review, Joshua Meyers!

            Show
            sullivan Ian Sullivan added a comment - Thank you for the quick review, Joshua Meyers !

              People

              • Assignee:
                sullivan Ian Sullivan
                Reporter:
                sullivan Ian Sullivan
                Reviewers:
                Joshua Meyers
                Watchers:
                Ian Sullivan, John Swinbank, Joshua Meyers
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel