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

Fix references to charImage.py in configs and make characterizeImage.py the default

    XMLWordPrintable

    Details

    • Story Points:
      0.1
    • Team:
      Architecture
    • Urgent?:
      No

      Description

      obs_subaru and obs_lsst both still load charImage.py in their config/processCcd.py. Although the latter is only a gen2 file, we should fix that, and fix any other places that refer to that file to have them point to config/characterizeImage.py, which is the actual task name. We also need to swap the contents of subaru's charImage and characterizeImage, as the latter is where the config should live going forward.

      This doesn't help configs that were written in gen2, but it will help us ensure that we have a chance of removing the config/charImage.py files at some point in the future.

      I believe subaru and lsst are the only obs packages with this problem, but we should try to survey for it elsewhere.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            It looks like obs_subaru has a charImage.py that is loaded by characterizeImage.py. obs_lsst does the reverse. Those are the only two charImage.py files remaining. I think this ticket require that obs_subaru charImage.py be renamed to characterizeImage.py.

            Show
            tjenness Tim Jenness added a comment - It looks like obs_subaru has a charImage.py that is loaded by characterizeImage.py. obs_lsst does the reverse. Those are the only two charImage.py files remaining. I think this ticket require that obs_subaru charImage.py be renamed to characterizeImage.py.
            Hide
            tjenness Tim Jenness added a comment -

            Trivial change. Jenkins is running. I'll do the obs_lsst deletion on DM-35531.

            Show
            tjenness Tim Jenness added a comment - Trivial change. Jenkins is running. I'll do the obs_lsst deletion on DM-35531 .
            Hide
            Parejkoj John Parejko added a comment -

            Thank you for cleaning this up.

            Show
            Parejkoj John Parejko added a comment - Thank you for cleaning this up.

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              Parejkoj John Parejko
              Reviewers:
              John Parejko
              Watchers:
              Eli Rykoff, John Parejko, Nate Lust, Tim Jenness, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins Builds

                  No builds found.