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

Improvements to processCcdWithFakes

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: pipe_tasks
    • Labels:
      None
    • Team:
      External

      Description

      I wanted to insert synthetic moving objects in HSC images with a catalog containing visit number, filters and magnitudes of a given object. I noticed that there is no option to just insert only star like objects. The default code requires that the catalog specify galaxy specific information (such as ellipticity, size, etc) which is odd. There is also no facility to input a given object into a particular visit.

       

      This improvement will 

      1) add an option, `insertOnlyStars` which will not require extraneous information to be passed in the fake object catalog.

      2) add an option which allows the user to inject objects belonging to a particular visit.

      3) Also deal with an issue where the fakes were not inserted at the edge CCDs of a field of view, as the pixscale on those CCDs was checked at the center of the FOV rather than the center of the CCD itself. Because the jointcal WCS has a reference at the center of the visit, this caused some fake insertions to not work correctly.

      I have a working branch for this code built on top of a recent weekly build. So if this sounds like something interesting to be added, this functionality can be added to the pipeline.

        Attachments

          Activity

          Hide
          yusra Yusra AlSayyad added a comment -

          I'm going to start the process of shepherding this external PR. It will take at least 7 hours to get through Jenkins, so Sophie Reed, you have some time to shout.

          The commit message is non-conformant, so I'm going to take the liberty to amend that after cherry-picking.

          Show
          yusra Yusra AlSayyad added a comment - I'm going to start the process of shepherding this external PR. It will take at least 7 hours to get through Jenkins, so Sophie Reed , you have some time to shout. The commit message is non-conformant, so I'm going to take the liberty to amend that after cherry-picking.
          Show
          yusra Yusra AlSayyad added a comment - Jenkins running here https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/35928/pipeline
          Hide
          yusra Yusra AlSayyad added a comment -

          Jenkins passed, and I got the green light to merge from Sophie Reed and Surhud More. Merging.

          Show
          yusra Yusra AlSayyad added a comment - Jenkins passed, and I got the green light to merge from Sophie Reed and Surhud More . Merging.
          Hide
          lskelvin Lee Kelvin added a comment -

          Marking ticket as reviewed. Code is being used on main at present, and is currently being ingested into the new source_injection repo.

          Show
          lskelvin Lee Kelvin added a comment - Marking ticket as reviewed. Code is being used on main at present, and is currently being ingested into the new source_injection repo.

            People

            Assignee:
            surhud Surhud More
            Reporter:
            surhud Surhud More
            Reviewers:
            Sophie Reed
            Watchers:
            Lee Kelvin, Sophie Reed, Surhud More, Yusra AlSayyad
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.