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

Warnings in test_camGeomFitsUtils.py

    XMLWordPrintable

    Details

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

      Description

      test_camGeomFitsUtils.py issues warnings that appear to be bugs we should fix. Examples:

        /Users/rowen/UW/LSST/lsstsw3/build/afw/python/lsst/afw/cameraGeom/fitsUtils.py:76: UserWarning: WARNING: Failed to set setName attribute with None value: setName(): incompatible function arguments. The following argument types are supported:
            1. (self: lsst.afw.table.ampInfo.ampInfo.AmpInfoRecord, name: str) -> None
      ...
        /Users/rowen/UW/LSST/lsstsw3/build/afw/python/lsst/afw/cameraGeom/fitsUtils.py:76: UserWarning: WARNING: Failed to set setBBox attribute with None value: setBBox(): incompatible function arguments. The following argument types are supported:
            1. (self: lsst.afw.table.ampInfo.ampInfo.AmpInfoRecord, bbox: lsst.afw.geom.box.Box2I) -> None
      

      See attached log for more info

        Attachments

          Issue Links

            Activity

            Hide
            rowen Russell Owen added a comment - - edited

            I think I have fixed the problem. Can you please carefully check to make sure the code still does what you intended? If you aren't sure then I'd rather hold off until we can figure out how to be certain.

            Show
            rowen Russell Owen added a comment - - edited I think I have fixed the problem. Can you please carefully check to make sure the code still does what you intended? If you aren't sure then I'd rather hold off until we can figure out how to be certain.
            Hide
            krughoff Simon Krughoff added a comment -

            Looks fine to me.

            Show
            krughoff Simon Krughoff added a comment - Looks fine to me.
            Hide
            jbosch Jim Bosch added a comment -

            This seems to be a duplicate of DM-12805, which I'll now close.  Thanks for fixing it!

            Show
            jbosch Jim Bosch added a comment - This seems to be a duplicate of DM-12805 , which I'll now close.  Thanks for fixing it!

              People

              Assignee:
              rowen Russell Owen
              Reporter:
              rowen Russell Owen
              Reviewers:
              Simon Krughoff
              Watchers:
              Jim Bosch, Russell Owen, Simon Krughoff
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.