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

Create new starSelector BaseClass

    Details

    • Story Points:
      4
    • Sprint:
      Alert Production S17 - 6, Alert Production F17 - 7, Alert Production F17 - 8
    • Team:
      Alert Production

      Description

      Implement new starSelector baseClass

        Attachments

          Issue Links

            Activity

            Hide
            cmorrison Chris Morrison added a comment -

            Started ticket to be worked on during UW pair coding day.

            Show
            cmorrison Chris Morrison added a comment - Started ticket to be worked on during UW pair coding day.
            Hide
            Parejkoj John Parejko added a comment -

            All selectors will be sourceSelectors and will inherit from BaseSourceSelector task, now that MakePsfCandidates has been removed from starSelector.run(), so we have no need for a starSelector-specific base class. Thus there will only be one registry.

            Note that this does not mean that all existing starSelectors will have to have their names changed: we can use the distinction between FooStarSelector and FooSourceSelector to identify selectors that are specifically designed to return psf-like sources.

            Show
            Parejkoj John Parejko added a comment - All selectors will be sourceSelectors and will inherit from BaseSourceSelector task, now that MakePsfCandidates has been removed from starSelector.run() , so we have no need for a starSelector-specific base class. Thus there will only be one registry. Note that this does not mean that all existing starSelectors will have to have their names changed: we can use the distinction between FooStarSelector and FooSourceSelector to identify selectors that are specifically designed to return psf-like sources.

              People

              • Assignee:
                cmorrison Chris Morrison
                Reporter:
                cmorrison Chris Morrison
                Watchers:
                Chris Morrison, John Parejko
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel