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

Rename phuInfo and infoList in obs_packages and base class

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Won't Fix
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: pipe_tasks
    • Labels:
      None

      Description

      As requested by John Parejko, please change the naming of phuInfo and infoList in pipe_tasks/ingest.py ParseTask.getInfo() method, and the naming where this is subclassed in the various obs_packages.

        Attachments

          Activity

          Hide
          lskelvin Lee Kelvin added a comment -

          John Parejko and Merlin Fisher-Levine, just following up on this ticket - can you explain what phuInfo and infoList should be changed to (and why)?

          Show
          lskelvin Lee Kelvin added a comment - John Parejko and Merlin Fisher-Levine , just following up on this ticket - can you explain what phuInfo and infoList should be changed to (and why)?
          Hide
          mfisherlevine Merlin Fisher-Levine added a comment -

          Oh boy, a trivial ticket from 4 years ago?! Errrr, I honestly can't remember this at all. Does Gen3 ingest even use that file, or is this about to be doubly irrelevant?

          John, do you remember why this mattered?

          Show
          mfisherlevine Merlin Fisher-Levine added a comment - Oh boy, a trivial ticket from 4 years ago?! Errrr, I honestly can't remember this at all. Does Gen3 ingest even use that file, or is this about to be doubly irrelevant? John, do you remember why this mattered?
          Hide
          tjenness Tim Jenness added a comment -

          It's irrelevant for gen3.

          Show
          tjenness Tim Jenness added a comment - It's irrelevant for gen3.
          Hide
          lskelvin Lee Kelvin added a comment -

          Thanks for that update Tim Jenness. Unless anyone objects, I'll mark this ticket as Won't Fix.

          Show
          lskelvin Lee Kelvin added a comment - Thanks for that update Tim Jenness . Unless anyone objects, I'll mark this ticket as Won't Fix .
          Hide
          mfisherlevine Merlin Fisher-Levine added a comment -

          Show
          mfisherlevine Merlin Fisher-Levine added a comment -
          Hide
          lskelvin Lee Kelvin added a comment -

          Marked as Won't Fix per the discussion above.

          Show
          lskelvin Lee Kelvin added a comment - Marked as Won't Fix per the discussion above.

            People

            Assignee:
            Unassigned Unassigned
            Reporter:
            mfisherlevine Merlin Fisher-Levine
            Watchers:
            John Parejko, Lee Kelvin, Merlin Fisher-Levine, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins Builds

                No builds found.