[v2] ipa: rpi: awb: Disable CT search bias for Grey World AWB
diff mbox series

Message ID 20241119101452.20808-1-naush@raspberrypi.com
State Accepted
Commit 934bbfce24f961c04132b8bb8bd66ea1ae999394
Headers show
Series
  • [v2] ipa: rpi: awb: Disable CT search bias for Grey World AWB
Related show

Commit Message

Naushir Patuck Nov. 19, 2024, 10:14 a.m. UTC 
If grey world AWB is setup in the tuning file, the CT curve will either
be missing or invalid. Disable biasing the statistics for the search in
such cases.

Fixes: ea8fd63d936f ("ipa: rpi: awb: Add a bias to the AWB search")
Signed-off-by: Naushir Patuck <naush@raspberrypi.com>
Reviewed-by: David Plowman <david.plowman@raspberrypi.com>
Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
---
 src/ipa/rpi/controller/rpi/awb.cpp | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

Patch
diff mbox series 

diff --git a/src/ipa/rpi/controller/rpi/awb.cpp b/src/ipa/rpi/controller/rpi/awb.cpp
index 9d8e170d1bfe..c277a1764112 100644
--- a/src/ipa/rpi/controller/rpi/awb.cpp
+++ b/src/ipa/rpi/controller/rpi/awb.cpp
@@ -459,10 +459,11 @@  void Awb::prepareStats()
 	 * LSC has already been applied to the stats in this pipeline, so stop
 	 * any LSC compensation.  We also ignore config_.fast in this version.
 	 */
+	const double biasCtR = config_.bayes ? config_.ctR.eval(config_.biasCT) : 0;
+	const double biasCtB = config_.bayes ? config_.ctB.eval(config_.biasCT) : 0;
 	generateStats(zones_, statistics_, config_.minPixels,
 		      config_.minG, getGlobalMetadata(),
-		      config_.biasProportion, config_.ctR.eval(config_.biasCT),
-		      config_.ctB.eval(config_.biasCT));
+		      config_.biasProportion, biasCtR, biasCtB);
 	/*
 	 * apply sensitivities, so values appear to come from our "canonical"
 	 * sensor.