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

Propagate flags to Object tables

    XMLWordPrintable

Details

    • Bug
    • Status: Done
    • Resolution: Done
    • None
    • DM
    • 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

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

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

          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/). jcarlin is it ok to merge when the Jenkins build passes? If not, what still needs to be done?

          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/ ). jcarlin is it ok to merge when the Jenkins build passes? If not, what still needs to be done?

          Jenkins build passed. Waiting on reviewer approval to merge.

          fred3m Fred Moolekamp added a comment - Jenkins build passed. Waiting on reviewer approval to merge.

          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.

          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.

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

          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

            fred3m Fred Moolekamp
            jcarlin Jeffrey Carlin
            Jeffrey Carlin
            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.