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

Typo in cp_pipe makeBrighterFatterKernel.py

    Details

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

      Description

      Two issues:

      (1) Function name in line 585 is mis-spelled.  Is spelled fitPtcAndNonlinearity and should be fitPtcAndNonLinearity.

      (2) Same function in line 585 is missing  the lookupTableArray input variable.

      I have verified that with these two fixes, it all runs OK.

      As an aside, I'm surprised there are not automated checks to check for function calls to functions that do not exist!

        Attachments

          Issue Links

            Activity

            cslage Craig Lage created issue -
            swinbank John Swinbank made changes -
            Field Original Value New Value
            Epic Link DM-22594 [ 427662 ]
            swinbank John Swinbank made changes -
            Team Data Release Production [ 10301 ]
            swinbank John Swinbank made changes -
            Link This issue relates to DM-23195 [ DM-23195 ]
            swinbank John Swinbank made changes -
            Link This issue relates to DM-23681 [ DM-23681 ]
            swinbank John Swinbank made changes -
            Story Points 1
            plazas Andrés Alejandro Plazas Malagón made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            plazas Andrés Alejandro Plazas Malagón made changes -
            Reviewers Merlin Fisher-Levine [ mfisherlevine ] John Swinbank [ swinbank ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            swinbank John Swinbank made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            plazas Andrés Alejandro Plazas Malagón made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]

              People

              • Assignee:
                plazas Andrés Alejandro Plazas Malagón
                Reporter:
                cslage Craig Lage
                Reviewers:
                John Swinbank
                Watchers:
                Andrés Alejandro Plazas Malagón, Craig Lage, John Swinbank
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Due:
                  Created:
                  Updated:
                  Resolved:

                  Summary Panel