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

Remove errant cout when reading old Calibs

    Details

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

      Description

      Looks like one of my debug statements from writing the PhotoCalib code to read old calibs got left in. This ticket is to remove it.

              std::cout << "!!!!!!!!!!PhotoCalib: " << calibration << " " << calibrationErr << std::endl;
      

        Attachments

          Activity

          Hide
          Parejkoj John Parejko added a comment -

          While looking into this, there are some other std::cout statements in afw that probably shouldn't be there.

          • GaussianProcess.cc 315, 327, 1991
          • Interpolate.cc 308

          In addition to some print functions that specifically go to std::cout instead of a generic stream.

          Show
          Parejkoj John Parejko added a comment - While looking into this, there are some other std::cout statements in afw that probably shouldn't be there. GaussianProcess.cc 315, 327, 1991 Interpolate.cc 308 In addition to some print functions that specifically go to std::cout instead of a generic stream.
          Hide
          Parejkoj John Parejko added a comment -

          Christopher Waters: can you review this trivial change?

          Github PR: https://github.com/lsst/afw/pull/450

          Show
          Parejkoj John Parejko added a comment - Christopher Waters : can you review this trivial change? Github PR: https://github.com/lsst/afw/pull/450
          Hide
          czw Christopher Waters added a comment -

          I even remember seeing that line, and thinking "huh, that's not needed" before it slipped my mind.

          Show
          czw Christopher Waters added a comment - I even remember seeing that line, and thinking "huh, that's not needed" before it slipped my mind.
          Hide
          Parejkoj John Parejko added a comment -

          Thanks for the quick review. Merged and done.

          Show
          Parejkoj John Parejko added a comment - Thanks for the quick review. Merged and done.

            People

            • Assignee:
              Parejkoj John Parejko
              Reporter:
              Parejkoj John Parejko
              Reviewers:
              Christopher Waters
              Watchers:
              Christopher Waters, Jim Bosch, John Parejko, Kian-Tat Lim
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel