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

Re-make Gaia refcat to fix epoch

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: meas_algorithms
    • Labels:
      None
    • Story Points:
      1
    • Sprint:
      AP F19-5 (October), AP F19-6 (November)
    • Team:
      Alert Production

      Description

      Siegfried Eggl just pointed out to me that the time epoch expressed in the gaia catalog are given in TCB, not UTC as I had thought. This results in a ~50s systematic offset of all of the epochs (all Gaia DR2 epochs are J2015.5). Although 50s probably doesn't matter for LSST parallax/PM calculations (which we're not doing yet anyway), it's easy enough to fix the ingest config and re-process the data. I can set that to run this weekend.

      While I'm at it, I will correct the meas_algorithms "how to make a refcat" guide to have the updated config too.

      Once it's generated and on lsst-dev, I'll have to re-request permission to copy it into place. Since the catalog is so new, I'll just overwrite the existing one entirely.

        Attachments

          Issue Links

            Activity

            Hide
            Parejkoj John Parejko added a comment -

            I did one run of singleFrameDriver on a COSMOS HSC-G tract. Output is here: /project/parejkoj/hscRerun/DM-22027. As before, some of the exposures failed to get an astrometry fit, which is likely due to configuration changes that are necessary to work with gaia.

            Is this sufficient?

            Show
            Parejkoj John Parejko added a comment - I did one run of singleFrameDriver on a COSMOS HSC-G tract. Output is here: /project/parejkoj/hscRerun/ DM-22027 . As before, some of the exposures failed to get an astrometry fit, which is likely due to configuration changes that are necessary to work with gaia. Is this sufficient?
            Hide
            price Paul Price added a comment -

            Sounds reasonable, thanks!

            Show
            price Paul Price added a comment - Sounds reasonable, thanks!
            Hide
            Parejkoj John Parejko added a comment -

            Paul Price: Did you have anything to say about the meas_algorithms PR?

            Show
            Parejkoj John Parejko added a comment - Paul Price : Did you have anything to say about the meas_algorithms PR?
            Hide
            price Paul Price added a comment -

            I hadn't seen it (sorry!), but I've looked now and it's fine. Thanks!

            Show
            price Paul Price added a comment - I hadn't seen it (sorry!), but I've looked now and it's fine. Thanks!
            Hide
            Parejkoj John Parejko added a comment -

            Thanks for the review (and sorry about the second review request: I clicked the wrong buttons when I meant to close it).

            Merged and done.

            Show
            Parejkoj John Parejko added a comment - Thanks for the review (and sorry about the second review request: I clicked the wrong buttons when I meant to close it). Merged and done.

              People

              Assignee:
              Parejkoj John Parejko
              Reporter:
              Parejkoj John Parejko
              Reviewers:
              Paul Price
              Watchers:
              John Parejko, John Swinbank, Paul Price, Siegfried Eggl, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins Builds

                  No builds found.