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

Move plotting IO out of plotting functions in coaddAnalysis

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: None
    • Labels:
    • Story Points:
      48
    • Epic Link:
    • Sprint:
      DRP F19-6 (Nov), DRP S20-2 (Jan), DRP S20-3 (Feb), DRP S20-5 (Apr), DRP S20-6 (May)
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      To convert pipe_analysis to gen 3 the butler calls need to be taken out of the lower level functions. As a first step move all the plotting related IO into runDataRef.

        Attachments

          Issue Links

            Activity

            Hide
            sophiereed Sophie Reed added a comment -

            I have checked all the scripts and they produce the same plots as on master.

            Show
            sophiereed Sophie Reed added a comment - I have checked all the scripts and they produce the same plots as on master.
            Hide
            yusra Yusra AlSayyad added a comment -

            Have a few questions and comments in github. In general, I think changing some return tuples to return Structs will go along with in future proofing it. 

            Show
            yusra Yusra AlSayyad added a comment - Have a few questions and comments in github. In general, I think changing some return tuples to return Structs will go along with in future proofing it. 
            Hide
            yusra Yusra AlSayyad added a comment -

            See comments on github.

            I'm not crazy about the readCatalogs' getUri, fitsReader I/O thing going on. We'll have to fix it for Gen 3, I'm ok with deferring until then.

            Show
            yusra Yusra AlSayyad added a comment - See comments on github. I'm not crazy about the readCatalogs' getUri, fitsReader I/O thing going on. We'll have to fix it for Gen 3, I'm ok with deferring until then.
            Hide
            sophiereed Sophie Reed added a comment -

            This was copied from Kian-Tat Lim's fixes to improve the IO issues. Hopefully long term it can run on top of the source tables/object tables and we can avoid a lot of this.

            Show
            sophiereed Sophie Reed added a comment - This was copied from Kian-Tat Lim 's fixes to improve the IO issues. Hopefully long term it can run on top of the source tables/object tables and we can avoid a lot of this.

              People

              Assignee:
              sophiereed Sophie Reed
              Reporter:
              sophiereed Sophie Reed
              Reviewers:
              Yusra AlSayyad
              Watchers:
              Nate Lust, Sophie Reed, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  CI Builds

                  No builds found.