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

obs_cfht is broken with the current stack

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: obs_cfht
    • Labels:
      None
    • Story Points:
      1
    • Sprint:
      Science Pipelines DM-S15-4, Science Pipelines DM-S15-5
    • Team:
      Alert Production

      Description

      obs_cfht's camera mapper is missing the new packageName class variable, so it is not compatible with the current stack.

      I suggest fixing obs_sdss and obs_subaru as well, if they need it.

        Attachments

          Issue Links

            Activity

            No builds found.
            rowen Russell Owen created issue -
            Hide
            tjenness Tim Jenness added a comment -

            Is this related to DM-2789 ?

            Show
            tjenness Tim Jenness added a comment - Is this related to DM-2789 ?
            Hide
            rowen Russell Owen added a comment -

            Yes, I believe so

            Show
            rowen Russell Owen added a comment - Yes, I believe so
            tjenness Tim Jenness made changes -
            Field Original Value New Value
            Link This issue is triggered by DM-2789 [ DM-2789 ]
            rowen Russell Owen made changes -
            Assignee Russell Owen [ rowen ]
            Hide
            rowen Russell Owen added a comment - - edited

            I have pushed fixes to obs_cfht and obs_subaru on tickets/DM-2910. These include a test of the new name.

            I tried running obs_cfht tests but it turns out they are broken. I filed DM-2917.

            Show
            rowen Russell Owen added a comment - - edited I have pushed fixes to obs_cfht and obs_subaru on tickets/ DM-2910 . These include a test of the new name. I tried running obs_cfht tests but it turns out they are broken. I filed DM-2917 .
            Hide
            rowen Russell Owen added a comment -

            Lauren: would you be willing to review these small changes? I figured you might even be able to build obs_subaru and run the tests (something I cannot do).

            Note: I won't merge until I manage to get the obs_cfht code run with testdata_cfht

            Show
            rowen Russell Owen added a comment - Lauren: would you be willing to review these small changes? I figured you might even be able to build obs_subaru and run the tests (something I cannot do). Note: I won't merge until I manage to get the obs_cfht code run with testdata_cfht
            rowen Russell Owen made changes -
            Reviewers Lauren MacArthur [ lauren ]
            Status To Do [ 10001 ] In Review [ 10004 ]
            rowen Russell Owen made changes -
            Story Points 1
            rowen Russell Owen made changes -
            Watchers Lauren MacArthur, Russell Owen, Tim Jenness [ Lauren MacArthur, Russell Owen, Tim Jenness ] Joshua Hoblitt, Lauren MacArthur, Russell Owen, Tim Jenness [ Joshua Hoblitt, Lauren MacArthur, Russell Owen, Tim Jenness ]
            Hide
            rowen Russell Owen added a comment -

            I added one more fix, which fits under the title, though it is a different problem than I originally had in mind when I wrote the ticket: config/processCcd.py override file uses the wrong path to get to the color term data.

            Show
            rowen Russell Owen added a comment - I added one more fix, which fits under the title, though it is a different problem than I originally had in mind when I wrote the ticket: config/processCcd.py override file uses the wrong path to get to the color term data.
            rowen Russell Owen made changes -
            Epic Link DM-2026 [ 16157 ]
            rowen Russell Owen made changes -
            Sprint Science Pipelines DM-S15-4 [ 159 ]
            Team Alert Production [ 10300 ]
            Hide
            lauren Lauren MacArthur added a comment -

            Sure, I'll have a look (after I finish a review for rhl, so may take a few days).

            Show
            lauren Lauren MacArthur added a comment - Sure, I'll have a look (after I finish a review for rhl, so may take a few days).
            Hide
            lauren Lauren MacArthur added a comment -

            I made some minor comments to a few commits on GitHub. Once addressed you are good to merge.

            Show
            lauren Lauren MacArthur added a comment - I made some minor comments to a few commits on GitHub. Once addressed you are good to merge.
            lauren Lauren MacArthur made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            swinbank John Swinbank made changes -
            Sprint Science Pipelines DM-S15-4 [ 159 ] Science Pipelines DM-S15-4, Science Pipelines DM-S15-5 [ 159, 162 ]
            swinbank John Swinbank made changes -
            Rank Ranked higher
            rowen Russell Owen made changes -
            Comment [ How do I find your comments? Normally I'd look for a pull request, but I don't see any. ]
            Hide
            rowen Russell Owen added a comment -

            Merged to master June 17

            Show
            rowen Russell Owen added a comment - Merged to master June 17
            rowen Russell Owen made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]

              People

              Assignee:
              rowen Russell Owen
              Reporter:
              rowen Russell Owen
              Reviewers:
              Lauren MacArthur
              Watchers:
              Joshua Hoblitt, Lauren MacArthur, Russell Owen, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins Builds

                  No builds found.