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

Port obs_ctio0m9 to gen3

    XMLWordPrintable

    Details

    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      If we intend to continue using obs_ctio0m9 and the related CI package, we need to port it to Gen3. This means:

      • Write astro_metadata_translator translator
      • Write an Instrument class and Formatter

      Please close as Won't Fix if no-one cares and people are okay with removing the obs package. I don't think we've formally deprecated it but I might be wrong. I'm assuming this ticket is not a blocker on DM-DAX-12.

        Attachments

          Issue Links

            Activity

            Hide
            mfisherlevine Merlin Fisher-Levine added a comment -

            I support closing this as Won't Fix, given that ci_cpp and other ci packages will be able to replace the ci_ctio_0m9 part, and that, IMO, is the only bit of this worth keeping. It might be good for John Swinbank and Christopher Waters to chime in too though.

            The only other reason to keep this would be to allow atmospec/Spectractor to (more) easily interface with the precursor CTIO data, but I personally don't think it's worth the hassle.

            Show
            mfisherlevine Merlin Fisher-Levine added a comment - I support closing this as Won't Fix, given that ci_cpp and other ci packages will be able to replace the ci_ctio_0m9 part, and that, IMO, is the only bit of this worth keeping. It might be good for John Swinbank and Christopher Waters to chime in too though. The only other reason to keep this would be to allow atmospec/Spectractor to (more) easily interface with the precursor CTIO data, but I personally don't think it's worth the hassle.
            Hide
            czw Christopher Waters added a comment -

            I have no objection.  The validation being run in the ci_ package seems to be very minimal (e.g. "is the output an exposure of non-zero size"), so it doesn't seem like there's much reason to keep it.

            Show
            czw Christopher Waters added a comment - I have no objection.  The validation being run in the ci_ package seems to be very minimal (e.g. "is the output an exposure of non-zero size"), so it doesn't seem like there's much reason to keep it.
            Hide
            tjenness Tim Jenness added a comment -

            It sounds like no-one wants this obs package to actually process the ctio0m9 data so dropping the package is fine with me.

            Show
            tjenness Tim Jenness added a comment - It sounds like no-one wants this obs package to actually process the ctio0m9 data so dropping the package is fine with me.
            Hide
            rhl Robert Lupton added a comment -

            Me too. I hope to never again look at CTIO 0.9m data.

            Show
            rhl Robert Lupton added a comment - Me too. I hope to never again look at CTIO 0.9m data.
            Hide
            swinbank John Swinbank added a comment -

            Show
            swinbank John Swinbank added a comment -
            Hide
            swinbank John Swinbank added a comment -
            Show
            swinbank John Swinbank added a comment - RFC-729 .
            Hide
            tjenness Tim Jenness added a comment -

            This ticket has been obsoleted by DM-26867.

            Show
            tjenness Tim Jenness added a comment - This ticket has been obsoleted by DM-26867 .

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              tjenness Tim Jenness
              Watchers:
              Christopher Waters, John Swinbank, Merlin Fisher-Levine, Robert Lupton, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.