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

Add Gen3 Instrument and Formatter classes for obs_lsst

    XMLWordPrintable

    Details

    • Story Points:
      4
    • Epic Link:
    • Sprint:
      BG3_F18_10, BG3_F18_11, BG3_S19_01, Arch 2019-07-15, Arch 2019-07-22, Arch 2019-07-29
    • Team:
      Architecture

      Description

      I anticipate this involving some changes to the Instrument base class, and hence changes to our only existing implementation of it in obs_subaru as well.

       

        Attachments

          Issue Links

          is blocked by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-20763 Add initial support for Gen3 Butler to obs_decam

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-20812 Rename TablePersistable storage classes in gen 3 to drop TablePersistable

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-21357 Add items() method to PropertySet/PropertyList

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-16412 Detectors obtained from Camera have wrong amplifer bboxes for untrimmed+assembled in obs_lsst

          • Won't Fix
          is triggering

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-20994 Write how to make gen3 obs package guide

          • Done
          mentioned in

          Page Loading...

            Activity

            Ascending order - Click to sort in descending order
            No builds found.
            jbosch Jim Bosch created issue -
            jbosch Jim Bosch made changes -
            Field Original Value New Value
            Epic Link DM-14448 [ 80390 ]
            jbosch Jim Bosch made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            jbosch Jim Bosch made changes -
            Link This issue is blocked by DM-16412 [ DM-16412 ]
            vaikunth Vaikunth Thukral made changes -
            Sprint BG3_F18_10 [ 797 ] BG3_F18_10, BG3_F18_11 [ 797, 812 ]
            yusra Yusra AlSayyad made changes -
            Epic Link DM-14448 [ 80390 ] DM-16675 [ 235235 ]
            vaikunth Vaikunth Thukral made changes -
            Sprint BG3_F18_10, BG3_F18_11 [ 797, 812 ] BG3_F18_10, BG3_F18_11, BG3_S19_01 [ 797, 812, 850 ]
            tjenness Tim Jenness made changes -
            Component/s obs_lsst [ 16504 ]
            Component/s obs_lsstCam [ 15215 ]
            vaikunth Vaikunth Thukral made changes -
            Sprint BG3_F18_10, BG3_F18_11, BG3_S19_01 [ 797, 812, 850 ] BG3_F18_10, BG3_F18_11, BG3_S19_01, BG3_S19_02 [ 797, 812, 850, 857 ]
            yusra Yusra AlSayyad made changes -
            Epic Link DM-16675 [ 235235 ] DM-16771 [ 235477 ]
            frossie Frossie Economou made changes -
            Status Admin Review [ 3 ] In Progress [ 11605 ]
            frossie Frossie Economou made changes -
            Status Review [ 11605 ] In Progress [ 3 ]
            yusra Yusra AlSayyad made changes -
            Epic Link DM-16771 [ 235477 ] DM-20166 [ 323037 ]
            tjenness Tim Jenness made changes -
            Assignee Jim Bosch [ jbosch ] Tim Jenness [ tjenness ]
            tjenness Tim Jenness made changes -
            Assignee Tim Jenness [ tjenness ] Jim Bosch [ jbosch ]
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Jim Bosch shall I take this ticket over so you can review it?

            Show
            tjenness Tim Jenness added a comment - Jim Bosch shall I take this ticket over so you can review it?
            Hide
            Permalink
            jbosch Jim Bosch added a comment -

            Please do.

            Show
            jbosch Jim Bosch added a comment - Please do.
            tjenness Tim Jenness made changes -
            Assignee Jim Bosch [ jbosch ] Tim Jenness [ tjenness ]
            tjenness Tim Jenness made changes -
            Team Data Release Production [ 10301 ] Architecture [ 10304 ]
            tjenness Tim Jenness made changes -
            Epic Link DM-20166 [ 323037 ] DM-20101 [ 319087 ]
            tjenness Tim Jenness made changes -
            Link This issue is blocked by DM-20812 [ DM-20812 ]
            tjenness Tim Jenness made changes -
            Sprint BG3_F18_10, BG3_F18_11, BG3_S19_01, BG3_S19_02 [ 797, 812, 850, 857 ] BG3_F18_10, BG3_F18_11, BG3_S19_01, Arch 2019-07-15 [ 797, 812, 850, 936 ]
            tjenness Tim Jenness made changes -
            Sprint BG3_F18_10, BG3_F18_11, BG3_S19_01, Arch 2019-07-15 [ 797, 812, 850, 936 ] BG3_F18_10, BG3_F18_11, BG3_S19_01, Arch 2019-07-15, Arch 2019-07-22 [ 797, 812, 850, 936, 937 ]
            tjenness Tim Jenness made changes -
            Sprint BG3_F18_10, BG3_F18_11, BG3_S19_01, Arch 2019-07-15, Arch 2019-07-22 [ 797, 812, 850, 936, 937 ] BG3_F18_10, BG3_F18_11, BG3_S19_01, Arch 2019-07-15, Arch 2019-07-22, Arch 2019-07-29 [ 797, 812, 850, 936, 937, 943 ]
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Jim Bosch this should probably be reviewed now so that we understand how it relates to DM-20763. John Parejko do you notice anything that will be affected by your ticket?

            All tests pass except the crosstalk assembly test which is marked as an expected failure and which we agreed on slack was outside the scope of this ticket.

            Show
            tjenness Tim Jenness added a comment - Jim Bosch this should probably be reviewed now so that we understand how it relates to DM-20763 . John Parejko do you notice anything that will be affected by your ticket? All tests pass except the crosstalk assembly test which is marked as an expected failure and which we agreed on slack was outside the scope of this ticket.
            tjenness Tim Jenness made changes -
            Reviewers Jim Bosch [ jbosch ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            Hide
            Permalink
            Parejkoj John Parejko added a comment -

            I suggest holding off on review of this until DM-20154 is merged, so that we can refactor it to use the new WCS code defined there. It might also be worth waiting for DM-20763, so we can use whatever cleanups of Instrument and Formatters come out of that.

            Show
            Parejkoj John Parejko added a comment - I suggest holding off on review of this until DM-20154 is merged, so that we can refactor it to use the new WCS code defined there. It might also be worth waiting for DM-20763 , so we can use whatever cleanups of Instrument and Formatters come out of that.
            Hide
            Permalink
            jbosch Jim Bosch added a comment -

            Fine with me; Tim Jenness, please ping me if you disagree or when those conditions are met.

            Show
            jbosch Jim Bosch added a comment - Fine with me; Tim Jenness , please ping me if you disagree or when those conditions are met.
            tjenness Tim Jenness made changes -
            Link This issue is blocked by DM-20763 [ DM-20763 ]
            Parejkoj John Parejko made changes -
            Link This issue is triggering DM-20994 [ DM-20994 ]
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Jim Bosch I think it's ready for another review. I've added the formatters and the instrument classes and cleaned up assembly some more to ensure gen2 and gen3 share code.

            Some open issues:

            • getCamera is now a class method because I want to cache the yaml camera given how long it takes to be created. I also see that we moved from a property to a getter when John Parejko refactored instrument a while back.
            • I'm a bit concerned about the requirement that detector_exposure_id in gen3 packer matches gen2 and matches astro_metadata_translator. The instrument classes now query astro_metadata_translator for max exposure id and max detector.
            Show
            tjenness Tim Jenness added a comment - Jim Bosch I think it's ready for another review. I've added the formatters and the instrument classes and cleaned up assembly some more to ensure gen2 and gen3 share code. Some open issues: getCamera is now a class method because I want to cache the yaml camera given how long it takes to be created. I also see that we moved from a property to a getter when John Parejko refactored instrument a while back. I'm a bit concerned about the requirement that detector_exposure_id in gen3 packer matches gen2 and matches astro_metadata_translator. The instrument classes now query astro_metadata_translator for max exposure id and max detector.
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Christopher Waters would you be able to do this review? Some of it does affect code you are working on.

            John Parejko would you please be able to look at the obs_base changes?

            Show
            tjenness Tim Jenness added a comment - Christopher Waters would you be able to do this review? Some of it does affect code you are working on. John Parejko would you please be able to look at the obs_base changes?
            tjenness Tim Jenness made changes -
            Reviewers Jim Bosch [ jbosch ] Christopher Waters [ cwaters ]
            Status In Review [ 10004 ] In Review [ 10004 ]
            tjenness Tim Jenness made changes -
            Link This issue is blocked by DM-21357 [ DM-21357 ]
            tjenness Tim Jenness made changes -
            Remote Link This issue links to "Page (Confluence)" [ 22001 ]
            Hide
            Permalink
            tjenness Tim Jenness added a comment - - edited

            Finally merged this.

            Christopher Waters sorry but you'll have to deal with a conflict.

            Show
            tjenness Tim Jenness added a comment - - edited Finally merged this. Christopher Waters sorry but you'll have to deal with a conflict.
            tjenness Tim Jenness made changes -
            Resolution Done [ 10000 ]
            Status In Review [ 10004 ] Done [ 10002 ]
            tjenness Tim Jenness made changes -
            Link This issue is triggering DM-31890 [ DM-31890 ]
            tjenness Tim Jenness made changes -
            Link This issue is triggering DM-31890 [ DM-31890 ]

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              jbosch Jim Bosch
              Reviewers:
              Christopher Waters
              Watchers:
              Christopher Waters, Hsin-Fang Chiang, Jim Bosch, John Parejko, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.