Uploaded image for project: 'Request For Comments'
  1. Request For Comments
  2. RFC-273

Planned change of arguments for single frame measurement run method

    Details

    • Type: RFC
    • Status: Implemented
    • Resolution: Done
    • Component/s: DM
    • Labels:
      None

      Description

      When the transition for meas_algorithms to meas_base was originally done, part of Jim's spec was to change the order of arguments in the run() method of SingleFrameMeasurementTask. But since we wanted the meas_algorithms framework to continue to work, we decided to temporarily allow both argument orders,

      As per DM-3804, we are now going to allow only the run(measCat, exposure).

      I intend to track down any contrary instances of this call in the stack, but I may miss things in private code or code outside of the main pipeline. Please inform me of any code which uses the sfm.run(exposure, measCat) ordering so I can fix it during this change.

        Attachments

          Container Issues

            Issue Links

              Activity

                People

                • Assignee:
                  pgee Perry Gee
                  Reporter:
                  pgee Perry Gee
                  Watchers:
                  Jim Bosch, John Swinbank, Perry Gee, Russell Owen, Tim Jenness, Yusra AlSayyad
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  6 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Planned End:

                    Summary Panel