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

Update pipe_analysis scripts for "filter" to "band" name change

    XMLWordPrintable

    Details

    • Story Points:
      2
    • Epic Link:
    • Sprint:
      DRP S21a (Dec Jan)
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      As of DM-27170, the level named "filter" in the MultilevelParquetTable s persisted by pipe_task s postprocess.py has changed to "band". Update the scripts that read in these tables accordingly.

        Attachments

          Activity

          Hide
          lauren Lauren MacArthur added a comment -

          Urghhh...this ended up being more involved than I'd hoped, but I think things are working now. Could you give this a spin (and effective review)? Note there is also an obs_subaru branch.
          I've done some checking on all scripts and they are all now passing for me for both pre- and post-filter->band parquet files (but your RC2 submission will prove a much more thorough check for the "current" status for the latter!)

          Show
          lauren Lauren MacArthur added a comment - Urghhh...this ended up being more involved than I'd hoped, but I think things are working now. Could you give this a spin (and effective review)? Note there is also an obs_subaru branch. I've done some checking on all scripts and they are all now passing for me for both pre- and post-filter->band parquet files (but your RC2 submission will prove a much more thorough check for the "current" status for the latter!)
          Hide
          emorganson Eric Morganson added a comment -

          Confirmed that coaddAnalysis and colorAnalysis both work now.

          Show
          emorganson Eric Morganson added a comment - Confirmed that coaddAnalysis and colorAnalysis both work now.
          Hide
          lauren Lauren MacArthur added a comment -

          Awesome.  Thanks, Eric!  Merged to master and done.

          Show
          lauren Lauren MacArthur added a comment - Awesome.  Thanks, Eric!  Merged to master and done.

            People

            Assignee:
            lauren Lauren MacArthur
            Reporter:
            lauren Lauren MacArthur
            Reviewers:
            Eric Morganson
            Watchers:
            Eric Morganson, Lauren MacArthur
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                CI Builds

                No builds found.