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

Refactor meas_extenstions/psfexStarSelector to new BaseClass

    XMLWordPrintable

    Details

      Attachments

        Issue Links

        is blocked by

        Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-9833 Create new sourceSelector BaseClass

        • Done

        Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-9835 Refactor astrometrySourceSelector to new BaseClass

        • Done
        is duplicated by

        Bug - A problem which impairs or prevents the functions of the product. DM-16899 meas_extensions_psfex has a psfex.py that seems to be pure bitrot

        • Won't Fix

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          Parejkoj John Parejko added a comment -

          Chris Morrison [X] and I are doing this as a pair coding, to finish off the epic.

          Show
          Parejkoj John Parejko added a comment - Chris Morrison [X] and I are doing this as a pair coding, to finish off the epic.
          Hide
          Permalink
          Parejkoj John Parejko added a comment -

          Simon Krughoff: you filed RFC-312 (make psfex the default), so I'm giving this to you to review.

          Note that it doesn't have any tests and doesn't appear to be used anywhere, so we can't guarantee that our changes work. The tests that do exist in meas_extensions_psfex use objectSize anyway, so this source selector may be entirely irrelevant. It's not documented, so we have no idea what it's trying to do.

          Show
          Parejkoj John Parejko added a comment - Simon Krughoff : you filed RFC-312 (make psfex the default), so I'm giving this to you to review. Note that it doesn't have any tests and doesn't appear to be used anywhere, so we can't guarantee that our changes work. The tests that do exist in meas_extensions_psfex use objectSize anyway, so this source selector may be entirely irrelevant. It's not documented, so we have no idea what it's trying to do.
          Hide
          Permalink
          swinbank John Swinbank added a comment -

          Hey folks, I found this ticket hanging out which has been in review with no further commentary for 6 months. Simon Krughoff, is this on your todo list?

          Show
          swinbank John Swinbank added a comment - Hey folks, I found this ticket hanging out which has been in review with no further commentary for 6 months. Simon Krughoff , is this on your todo list?
          Hide
          Permalink
          Parejkoj John Parejko added a comment - - edited

          Pinging this one again Simon Krughoff: what do we do with it? You haven't commented on the PR.

          Show
          Parejkoj John Parejko added a comment - - edited Pinging this one again Simon Krughoff : what do we do with it? You haven't commented on the PR.
          Hide
          Permalink
          krughoff Simon Krughoff added a comment -

          I'm very sorry about letting this go. I'll get on it today.

          Show
          krughoff Simon Krughoff added a comment - I'm very sorry about letting this go. I'll get on it today.
          Hide
          Permalink
          krughoff Simon Krughoff added a comment -

          The question was what do we do with this. I suggest merging and closing this ticket. We should take a pass for untested code at some point in the relatively near future, but I don't see why that should hold this up.

          Show
          krughoff Simon Krughoff added a comment - The question was what do we do with this. I suggest merging and closing this ticket. We should take a pass for untested code at some point in the relatively near future, but I don't see why that should hold this up.
          Hide
          Permalink
          Parejkoj John Parejko added a comment -

          Thanks Simon Krughoff. Note that I messed up the master rebase somehow, resulting in PR 30 becoming a no-op. John Swinbank helped me recover from the reflog, and I made a new PR (43) and merged that.

          Show
          Parejkoj John Parejko added a comment - Thanks Simon Krughoff . Note that I messed up the master rebase somehow, resulting in PR 30 becoming a no-op. John Swinbank helped me recover from the reflog, and I made a new PR (43) and merged that.

            People

            Assignee:
            cmorrison Chris Morrison [X] (Inactive)
            Reporter:
            cmorrison Chris Morrison [X] (Inactive)
            Reviewers:
            Simon Krughoff
            Watchers:
            Chris Morrison [X] (Inactive), John Parejko, John Swinbank, Simon Krughoff
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.