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

Propagate flags to Object tables

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: DM
    • Labels:
      None

      Description

      Many flags and relevant columns are not currently being propagated to the Object tables. Update the relevant YAML definition (e.g., https://github.com/lsst/obs_subaru/blob/master/policy/Object.yaml#L318-L320) to include the desired columns.

      Columns to add:

      detect_isIsolated,
      detect_isDeblendedModelSource,
      detect_isDeblendedSource,
      detect_fromBlend
      merge_peak_sky

      (more to be added in comments?)

        Attachments

          Activity

          Hide
          fred3m Fred Moolekamp added a comment -

          Created DM-30353 and DM-20354 for creating the `isSky` column and adding `merge_measurement` column respectively.

          Show
          fred3m Fred Moolekamp added a comment - Created DM-30353 and DM-20354 for creating the `isSky` column and adding `merge_measurement` column respectively.
          Hide
          fred3m Fred Moolekamp added a comment -

          Jenkins build is successful on centos-7 but I'm still waiting on mac-os (https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/34306/pipeline/56/). Jeffrey Carlin is it ok to merge when the Jenkins build passes? If not, what still needs to be done?

          Show
          fred3m Fred Moolekamp added a comment - Jenkins build is successful on centos-7 but I'm still waiting on mac-os ( https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/34306/pipeline/56/ ). Jeffrey Carlin is it ok to merge when the Jenkins build passes? If not, what still needs to be done?
          Hide
          fred3m Fred Moolekamp added a comment -

          Jenkins build passed. Waiting on reviewer approval to merge.

          Show
          fred3m Fred Moolekamp added a comment - Jenkins build passed. Waiting on reviewer approval to merge.
          Hide
          yusra Yusra AlSayyad added a comment -

          If you merge this, I'm just going to change Merge_peak_sky back to merge_peak_sky behind you. Users aren't going to remember that there's one column that starts with a capital.

          Show
          yusra Yusra AlSayyad added a comment - If you merge this, I'm just going to change Merge_peak_sky back to merge_peak_sky behind you. Users aren't going to remember that there's one column that starts with a capital.
          Hide
          yusra Yusra AlSayyad added a comment -

          Since I'm on the west coast, I'm happy to fix it on this ticket, re-Jenkins and merge it before midnight.

          Show
          yusra Yusra AlSayyad added a comment - Since I'm on the west coast, I'm happy to fix it on this ticket, re-Jenkins and merge it before midnight.

            People

            Assignee:
            fred3m Fred Moolekamp
            Reporter:
            jcarlin Jeffrey Carlin
            Reviewers:
            Jeffrey Carlin
            Watchers:
            Fred Moolekamp, Jeffrey Carlin, Lauren MacArthur, Lee Kelvin, Yusra AlSayyad
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.