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

Cleanup fallout from removing interpolated from badFlags list

    XMLWordPrintable

    Details

    • Type: Story
    • Status: To Do
    • Resolution: Unresolved
    • Fix Version/s: None
    • Component/s: meas_algorithms
    • Labels:
      None

      Description

      DM-13493 removed interpolated_flag from the badFlags list in BaseSourceSelectorTask, but did not remove it from BaseStarSelectorTask. Removing it from starSelector results in the dm_stack_demo failing because the sources change. In order to finish unifying the starSelectors, we have do decide how to properly manage this change.

      One option would be to append interpolated_flag to badFlags in objectSizeSS and any other "only select PSF sources" starSelectors (i.e. "true" starSelectors, instead of sourceSelectors). That at least would keep the status quo.

        Attachments

          Issue Links

            Activity

            No builds found.
            Parejkoj John Parejko created issue -
            Parejkoj John Parejko made changes -
            Field Original Value New Value
            Epic Link DM-9832 [ 31214 ]
            Parejkoj John Parejko made changes -
            Link This issue relates to DM-13493 [ DM-13493 ]
            Parejkoj John Parejko made changes -
            Risk Score 0
            Parejkoj John Parejko made changes -
            Watchers Chris Morrison, Jim Bosch, John Parejko, John Swinbank [ Chris Morrison, Jim Bosch, John Parejko, John Swinbank ] Chris Morrison, Jim Bosch, John Parejko, John Swinbank, Lauren MacArthur [ Chris Morrison, Jim Bosch, John Parejko, John Swinbank, Lauren MacArthur ]
            Hide
            Parejkoj John Parejko added a comment -

            Lauren MacArthur found the relevant discussion:

            https://lsstc.slack.com/archives/C2JPL2DGD/p1525206128000555

            It appears that the best current behavior is for PSFy starSelectors to incorporate interpolated in the badFlags list, but generic sourceSelectors to not.

            Show
            Parejkoj John Parejko added a comment - Lauren MacArthur found the relevant discussion: https://lsstc.slack.com/archives/C2JPL2DGD/p1525206128000555 It appears that the best current behavior is for PSFy starSelectors to incorporate interpolated in the badFlags list, but generic sourceSelectors to not.
            swinbank John Swinbank made changes -
            Epic Link DM-9832 [ 31214 ] DM-16722 [ 235355 ]
            swinbank John Swinbank made changes -
            Epic Link DM-16722 [ 235355 ] DM-17887 [ 240317 ]
            swinbank John Swinbank made changes -
            Epic Link DM-17887 [ 240317 ] DM-19979 [ 307530 ]
            swinbank John Swinbank made changes -
            Epic Link DM-19979 [ 307530 ] DM-21441 [ 423048 ]
            Parejkoj John Parejko made changes -
            Link This issue relates to RFC-645 [ RFC-645 ]
            swinbank John Swinbank made changes -
            Epic Link DM-21441 [ 423048 ] DM-22484 [ 427311 ]
            swinbank John Swinbank made changes -
            Epic Link DM-22484 [ 427311 ] DM-24339 [ 433026 ]
            swinbank John Swinbank made changes -
            Labels SciencePipelines
            swinbank John Swinbank made changes -
            Team Alert Production [ 10300 ]
            Urgent? off
            swinbank John Swinbank made changes -
            Summary Cleanup stack demo and other fallout from removing interpolated from badFlags list Cleanup fallout from removing interpolated from badFlags list
            Hide
            swinbank John Swinbank added a comment -

            Removed mention of the stack demo from the summary of this ticket, since it's going away (per RFC-645). Not checked to see how much other fallout there might be.

            Show
            swinbank John Swinbank added a comment - Removed mention of the stack demo from the summary of this ticket, since it's going away (per RFC-645 ). Not checked to see how much other fallout there might be.
            swinbank John Swinbank made changes -
            Epic Link DM-24339 [ 433026 ] DM-25139 [ 435257 ]
            swinbank John Swinbank made changes -
            Epic Link DM-25139 [ 435257 ] DM-26810 [ 439762 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-26810 [ 439762 ] DM-27906 [ 442554 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-27906 [ 442554 ] DM-29214 [ 459218 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-29214 [ 459218 ] DM-30436 [ 504824 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-30436 [ 504824 ] DM-30501 [ 510159 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-30501 [ 510159 ] DM-30502 [ 510160 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-30502 [ 510160 ] DM-30506 [ 510172 ]
            horvat Nikolina Horvat made changes -
            Epic Link DM-30506 [ 510172 ] DM-34931 [ 1598492 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-34931 [ 1598492 ] DM-36006 [ 1997396 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-36006 [ 1997396 ] DM-36522 [ 2254239 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-36522 [ 2254239 ] DM-36523 [ 2254240 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-36523 [ 2254240 ] DM-36524 [ 2254243 ]
            sullivan Ian Sullivan made changes -
            Epic Link DM-36524 [ 2254243 ] DM-36525 [ 2254245 ]

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              Parejkoj John Parejko
              Watchers:
              Chris Morrison [X] (Inactive), Jim Bosch, John Parejko, John Swinbank, Lauren MacArthur
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:

                  Jenkins

                  No builds found.