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

Unhelpful error message from PessimisticPatternMatcherB

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: meas_astrom
    • Labels:
      None

      Description

      The call PessimisticPatternMatcherB(ref_array[:, :3], self.log) in matchPessimisticB.py around line 387 generates the error message

      ValueError: negative dimensions are not allowed

      when ref_array is empty. Next time someone's in the code, could we improve the error message?  It happens when there are no possible matches from the reference catalogue.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          swinbank John Swinbank added a comment -

          Hey Chris, do you have time for a tiny review?

          PR: https://github.com/lsst/meas_astrom/pull/136
          Jenkins: https://ci.lsst.codes/job/stack-os-matrix/32040/

          Show
          swinbank John Swinbank added a comment - Hey Chris, do you have time for a tiny review? PR: https://github.com/lsst/meas_astrom/pull/136 Jenkins: https://ci.lsst.codes/job/stack-os-matrix/32040/
          Hide
          Permalink
          swinbank John Swinbank added a comment -

          Re-Jenkins: https://ci.lsst.codes/job/stack-os-matrix/32100/

          Show
          swinbank John Swinbank added a comment - Re-Jenkins: https://ci.lsst.codes/job/stack-os-matrix/32100/
          Hide
          Permalink
          swinbank John Swinbank added a comment -

          Re-re-Jenkins: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/32102/

          Show
          swinbank John Swinbank added a comment - Re-re-Jenkins: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/32102/
          Hide
          Permalink
          swinbank John Swinbank added a comment -

          Thanks Chris!

          Show
          swinbank John Swinbank added a comment - Thanks Chris!

            People

            Assignee:
            swinbank John Swinbank
            Reporter:
            rhl Robert Lupton
            Reviewers:
            Chris Morrison [X] (Inactive)
            Watchers:
            Chris Morrison [X] (Inactive), John Swinbank, Robert Lupton
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.