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

Change CalibrateTask refcat defaults to Gaia DR2 for astrometry and PS1 for photometry

    XMLWordPrintable

Details

    • 4
    • AP S22-3 (February), AP S22-4 (March), AP S22-5 (April)
    • Alert Production
    • No

    Description

      This is the implementation ticket for RFC-697, to update the CalibrateTask reference catalog defaults to use Gaia DR2 for astrometry and PS1 for photometry and remove relevant overrides from obs packages. These are the defaults that we want, although exactly how to implement them (probably inside CalibrateConfig.setDefaults()?):

      astromRefObjLoader.ref_dataset_name = "gaia_dr2_20200414"
      astromRefObjLoader.anyFilterMapsToThis = "phot_g_mean"
      photoRefObjLoader.ref_dataset_name = "ps1_pv3_3pi_20170110"
      

      Attachments

        Issue Links

          Activity

            Parejkoj John Parejko added a comment -

            DM-27858 is the relevant ticket for validating the HSC changes. I don't know who is going to be responsible for that, but I'll bring it up on slack.

            Parejkoj John Parejko added a comment - DM-27858 is the relevant ticket for validating the HSC changes. I don't know who is going to be responsible for that, but I'll bring it up on slack.
            Parejkoj John Parejko added a comment -

            Thank you lskelvin for investigating this ticket on DECam. I've merged all the branches, but closed the obs_subaru PR without merging it.

            Parejkoj John Parejko added a comment - Thank you lskelvin for investigating this ticket on DECam. I've merged all the branches, but closed the obs_subaru PR without merging it.
            Parejkoj John Parejko added a comment -

            I neglected to re-run Jenkins after rebasing everything, and my merge broke the ap_verify package. I've put a fix for that on tickets/DM-27013-fix, but there is apparently a separate breakage when running ap_verify; I filed DM-34491 for that one.

            Parejkoj John Parejko added a comment - I neglected to re-run Jenkins after rebasing everything, and my merge broke the ap_verify package. I've put a fix for that on tickets/ DM-27013 -fix , but there is apparently a separate breakage when running ap_verify; I filed DM-34491 for that one.
            lskelvin Lee Kelvin added a comment -

            Thanks for the update John. For completeness, the fix PR is at this link.

            lskelvin Lee Kelvin added a comment - Thanks for the update John. For completeness, the fix PR is at this link .
            Parejkoj John Parejko added a comment -

            And one final PR to fix a pipelines_check breakage that was masked by the unmerged obs_subaru branch: https://github.com/lsst/obs_subaru/pull/416

            Parejkoj John Parejko added a comment - And one final PR to fix a pipelines_check breakage that was masked by the unmerged obs_subaru branch: https://github.com/lsst/obs_subaru/pull/416

            People

              Parejkoj John Parejko
              Parejkoj John Parejko
              Lee Kelvin
              Ian Sullivan, John Parejko, Krzysztof Findeisen, Lauren MacArthur, Lee Kelvin, Meredith Rawls, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.