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

Expand docs for AP HSC refcats

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ap_pipe
    • Labels:
      None
    • Story Points:
      0.5
    • Sprint:
      AP F22-5 (October)
    • Team:
      Alert Production
    • Urgent?:
      No

      Description

      Following on from our discussion at today's AP pipe meeting, we need to update the docs on our HSC calibrate.py config to clarify that PS1 is going to remain the most appropriate refcat for HSC astrometry, given the outcome of an inadvertent test of DM-27858.

      While there, I'll cleanup the filterMap loading, too.

        Attachments

          Issue Links

            Activity

            Hide
            Parejkoj John Parejko added a comment - - edited

            Jenkins: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/37433/pipeline/

            Doing a local ap_verify_ci_cosmos_pdr2 run to test the tweaked config file.

            Show
            Parejkoj John Parejko added a comment - - edited Jenkins: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/37433/pipeline/ Doing a local ap_verify_ci_cosmos_pdr2 run to test the tweaked config file.
            Hide
            Parejkoj John Parejko added a comment -

            Local run of ap_verify_ci_cosmos_pdr2 produced identical values to main, so I think we're good here.

            Show
            Parejkoj John Parejko added a comment - Local run of ap_verify_ci_cosmos_pdr2 produced identical values to main, so I think we're good here.
            Hide
            Parejkoj John Parejko added a comment -

            Meredith Rawls: Instead of assigning the ticket to you, you get to review it. It's very short.

            PR: https://github.com/lsst/ap_pipe/pull/126

            Show
            Parejkoj John Parejko added a comment - Meredith Rawls : Instead of assigning the ticket to you, you get to review it. It's very short. PR: https://github.com/lsst/ap_pipe/pull/126
            Hide
            krzys Krzysztof Findeisen added a comment - - edited

            While you're at it, can you address https://github.com/lsst/ap_pipe/pull/124#discussion_r952026025? I haven't had a chance to work on ap_pipe since that discussion.

            (Oops, just saw you put it into review )

            Show
            krzys Krzysztof Findeisen added a comment - - edited While you're at it, can you address https://github.com/lsst/ap_pipe/pull/124#discussion_r952026025? I haven't had a chance to work on ap_pipe since that discussion. (Oops, just saw you put it into review )
            Hide
            Parejkoj John Parejko added a comment -

            Krzysztof Findeisen: good idea. I've deleted that file now.

            Show
            Parejkoj John Parejko added a comment - Krzysztof Findeisen : good idea. I've deleted that file now.

              People

              Assignee:
              Parejkoj John Parejko
              Reporter:
              Parejkoj John Parejko
              Reviewers:
              Meredith Rawls
              Watchers:
              Ian Sullivan, John Parejko, Krzysztof Findeisen, Meredith Rawls
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.