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

Add explicit support for dataset name to refcat API

    XMLWordPrintable

    Details

    • Story Points:
      2

      Description

      Currently, various code assumes the refObjLoader has a `ref_dataset_name` field, but this is only true for the HTM Indexed refcats. We should make the dataset name an explicit part of the LoadReferenceObjects API (perhaps getDatasetName?) as an abstract method that the Indexed loader can then override to return that config value.

      This will still be a broken API for a.net refcats, but at least it will be more obviously so.

        Attachments

          Issue Links

          duplicates

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-35871 Add refcat name arg to ReferenceObjectLoader init

          • Done
          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-17029 Update LoadReferenceObjectsTask to output fluxes in nanojansky

          • Done

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-562 Drop support for astrometry.net reference catalogs with Gen3 middleware migration

          • Implemented

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-575 Convert old refcats to nJy for RFC-549

          • Implemented

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            dtaranu Dan Taranu added a comment -

            John Parejko is this still relevant? I thought we had dropped all support for a.net.

            Show
            dtaranu Dan Taranu added a comment - John Parejko is this still relevant? I thought we had dropped all support for a.net.
            Hide
            Permalink
            Parejkoj John Parejko added a comment -

            Although we have dropped support for a.net, I do think this would still be useful: it is helpful for extracting the correct colorterms, for instance. Maybe this should be tagged as a post-gen3 thing, as there is a new refcat system there?

            Show
            Parejkoj John Parejko added a comment - Although we have dropped support for a.net, I do think this would still be useful: it is helpful for extracting the correct colorterms, for instance. Maybe this should be tagged as a post-gen3 thing, as there is a new refcat system there?
            Hide
            Permalink
            Parejkoj John Parejko added a comment -

            I did this work on DM-35871, and didn't know about this ticket.

            Show
            Parejkoj John Parejko added a comment - I did this work on DM-35871 , and didn't know about this ticket.

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              Parejkoj John Parejko
              Watchers:
              Colin Slater, Dan Taranu, Eli Rykoff, John Parejko, John Swinbank, Paul Price, Russell Owen, Simon Krughoff, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              9 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins Builds

                  No builds found.