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

meas_algorithms warning message looks wrong if only 1 sources

    XMLWordPrintable

    Details

      Description

      As shown in DM-11779, we get warning messages from aperture correction looking like:

      only 1 sources, but require at least 2.

      This should be fixed to use correct grammar. The relevant code is python/lsst/meas/algorithms/measureApCorr.py lines 244-249. It looks to me like the warning message is constructed twice, and it would be cleaner to construct it once and use it twice.

        Attachments

          Issue Links

          is triggered by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-11779 Different processCcd outcomes using Py2 and Py3 stacks

          • Done
          mentioned in

          Page Loading...

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            It might be simpler to reformulate the message as:

            Number of sources: 1; required: 2

            Show
            tjenness Tim Jenness added a comment - It might be simpler to reformulate the message as: Number of sources: 1; required: 2
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            This has not been fixed but is still relevant.

            Show
            tjenness Tim Jenness added a comment - This has not been fixed but is still relevant.
            Hide
            Permalink
            yusra Yusra AlSayyad added a comment -

            Flagging this as an easy ticket for teaching the dev guide workflow

            Show
            yusra Yusra AlSayyad added a comment - Flagging this as an easy ticket for teaching the dev guide workflow

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              tjenness Tim Jenness
              Watchers:
              Tim Jenness, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:

                  Jenkins

                  No builds found.