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

Write patchId column in WriteObjectTableTask, rather than WriteQATableTask

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Story Points:
      0.25
    • Epic Link:
    • Team:
      Data Release Production

      Description

      Sophie Reed requests that patchId be a column in the deepCoadd_obj table, which makes sense (I'm currently creating it in writeQATableTask, but seems sensible that users might want it earlier).

        Attachments

          Activity

          Hide
          tmorton Tim Morton [X] (Inactive) added a comment -

          Sophie Reed OK, I think this should do what you want? If you could test it by running writeObjectTable.py and making sure all the columns you want (patchId, tractId) are there, that would be great.

          Show
          tmorton Tim Morton [X] (Inactive) added a comment - Sophie Reed OK, I think this should do what you want? If you could test it by running writeObjectTable.py and making sure all the columns you want ( patchId , tractId ) are there, that would be great.
          Hide
          sophiereed Sophie Reed added a comment -

          Thank you for the quick turnaround, the code looks fine and works. Only thing is that the white space changes and code changes should probably be on different commits.

          Show
          sophiereed Sophie Reed added a comment - Thank you for the quick turnaround, the code looks fine and works. Only thing is that the white space changes and code changes should probably be on different commits.
          Hide
          yusra Yusra AlSayyad added a comment -

          Tim Morton [X] Did this ticket get subsumed into DM-14289?

          Show
          yusra Yusra AlSayyad added a comment - Tim Morton [X] Did this ticket get subsumed into DM-14289 ?
          Hide
          tmorton Tim Morton [X] (Inactive) added a comment -

          Yusra AlSayyad, yes, that's correct. The patchId and tractId columns are in the current WriteObjectTableTask.

          Show
          tmorton Tim Morton [X] (Inactive) added a comment - Yusra AlSayyad , yes, that's correct. The patchId and tractId columns are in the current WriteObjectTableTask .

            People

            Assignee:
            tmorton Tim Morton [X] (Inactive)
            Reporter:
            tmorton Tim Morton [X] (Inactive)
            Reviewers:
            Sophie Reed
            Watchers:
            Sophie Reed, Tim Morton [X] (Inactive), Yusra AlSayyad
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.