Details

      Description

      Please add a table to store the final guide offset from the nominal position calculated after each AGC exposure is done. The current proposal is

          agc_exposure_id = Column(Integer, ForeignKey('agc_exposure.agc_exposure_id'),
                                   primary_key=True, autoincrement=False,
                                   comment='AGC exposure number identifier')
          guide_ra = Column(FLOAT,
                            comment='The calculated FoV R.A. center [deg.]')
          guide_dec = Column(FLOAT,
                             comment='The calculated FoV Dec. center [deg.]')
          guide_pa = Column(FLOAT,
                            comment='The calculated FoV PA [deg.]')
          guide_delta_ra = Column(FLOAT,
                                  comment='The calculated FoV R.A. offset [arcsec.]')
          guide_delta_dec = Column(FLOAT,
                                   comment='The calculated FoV Dec. offset [arcsec.]')
          guide_delta_insrot = Column(FLOAT,
                                      comment='The calculated InsRot offset [arcsec.]')
          guide_delta_az = Columnn(FLOAT,
                                   comment='The calculated Az offset [deg.] (optional)')
          guide_delta_el = Columnn(FLOAT,
                                   comment='The calculated El offset [deg.] (optional)')
          guide_delta_z = Column(REAL,
                                 comment='The calculated focus offset [mm]')
          guide_delta_z1 = Column(REAL,
                                  comment='The calculated focus offset for AGC1 [mm]')
          guide_delta_z2 = Column(REAL,
                                  comment='The calculated focus offset for AGC2 [mm]')
          guide_delta_z3 = Column(REAL,
                                  comment='The calculated focus offset for AGC3 [mm]')
          guide_delta_z4 = Column(REAL,
                                  comment='The calculated focus offset for AGC4 [mm]')
          guide_delta_z5 = Column(REAL,
                                  comment='The calculated focus offset for AGC5 [mm]')
          guide_delta_z6 = Column(REAL,
                                  comment='The calculated focus offset for AGC6 [mm]')
      

      We need the focus offset for all AGCs (delta_zx) and also the typical (mean?) value (delta_z).

        Attachments

          Activity

          Hide
          kiyoto.yabe Kiyoto Yabe added a comment -

          I have pushed the latest fix. Could you review the changes please?

          Show
          kiyoto.yabe Kiyoto Yabe added a comment - I have pushed the latest fix. Could you review the changes please?
          Hide
          hiro Yoshida, Hiroshige added a comment -

          guide_ra, guide_dec, guide_pa are intended to store design values. So the calculated center of FoV would be:

          (RA, DEC, PA) = (guide_ra + guide_delta_ra / 3600, guide_dec + guide_delta_dec / 3600, guide_pa + guide_delta_pa / 3600)

          As long as another table with agc_exposure_id as a primary key (ideally agc_exposure) has timestamps (mid-times of AGC exposures) stored, I think the agc_guide_offset table can do away with a timestamp column.

          Show
          hiro Yoshida, Hiroshige added a comment - guide_ra, guide_dec, guide_pa are intended to store design values. So the calculated center of FoV would be: (RA, DEC, PA) = (guide_ra + guide_delta_ra / 3600, guide_dec + guide_delta_dec / 3600, guide_pa + guide_delta_pa / 3600) As long as another table with agc_exposure_id as a primary key (ideally agc_exposure) has timestamps (mid-times of AGC exposures) stored, I think the agc_guide_offset table can do away with a timestamp column.
          Hide
          kiyoto.yabe Kiyoto Yabe added a comment -

          OK, I'll change the column comments for guide_ra/dec/pa.

          Yes, I hope we do not need timestamps in this table.

          Show
          kiyoto.yabe Kiyoto Yabe added a comment - OK, I'll change the column comments for guide_ra/dec/pa. Yes, I hope we do not need timestamps in this table.
          Hide
          kiyoto.yabe Kiyoto Yabe added a comment -

          merged and will close

          Show
          kiyoto.yabe Kiyoto Yabe added a comment - merged and will close

            People

            • Assignee:
              kiyoto.yabe Kiyoto Yabe
              Reporter:
              kiyoto.yabe Kiyoto Yabe
              Reviewers:
              Yoshida, Hiroshige
            • Votes:
              0 Vote for this issue
              Watchers:
              Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: