[PATCH v2 16/17] libipa: awb_bayes: Remove overly verbose log messages
Stefan Klug
stefan.klug at ideasonboard.com
Thu Jan 23 12:41:06 CET 2025
Logging every search step is too verbose even with debug messages
enabled and it hides the more important messages (min max values of
errors and likelihoods). Remove the debug messages in a separate commit,
so that it can easily be reverted if needed.
Signed-off-by: Stefan Klug <stefan.klug at ideasonboard.com>
---
Changes in v2:
- Added this commit
---
src/ipa/libipa/awb_bayes.cpp | 13 -------------
1 file changed, 13 deletions(-)
diff --git a/src/ipa/libipa/awb_bayes.cpp b/src/ipa/libipa/awb_bayes.cpp
index aaa8c7a663ad..6b88aebeffb5 100644
--- a/src/ipa/libipa/awb_bayes.cpp
+++ b/src/ipa/libipa/awb_bayes.cpp
@@ -330,12 +330,6 @@ double AwbBayes::coarseSearch(const ipa::Pwl &prior, const AwbStats &stats) cons
errorLimits.record(delta2Sum);
priorLogLikelihoodLimits.record(priorLogLikelihood);
- LOG(Awb, Debug) << "Coarse search t: " << t
- << " gains: " << gains
- << " error: " << delta2Sum
- << " prior: " << priorLogLikelihood
- << " likelihood: " << finalLogLikelihood;
-
points.push_back({ { t, finalLogLikelihood } });
if (points.back().y() < points[bestPoint].y())
bestPoint = points.size() - 1;
@@ -453,13 +447,6 @@ void AwbBayes::fineSearch(double &t, double &r, double &b, ipa::Pwl const &prior
double delta2Sum = stats.computeColourError(gains);
errorLimits.record(delta2Sum);
double finalLogLikelihood = delta2Sum - priorLogLikelihood;
- LOG(Awb, Debug)
- << "Fine search t: " << tTest
- << " r: " << rbTest[0]
- << " b: " << rbTest[1]
- << " offset: " << bestOffset
- << " likelihood: " << finalLogLikelihood
- << (finalLogLikelihood < bestLogLikelihood ? " NEW BEST" : "");
if (bestT == 0 || finalLogLikelihood < bestLogLikelihood) {
bestLogLikelihood = finalLogLikelihood;
--
2.43.0
More information about the libcamera-devel
mailing list