[{"id":27266,"web_url":"https://patchwork.libcamera.org/comment/27266/","msgid":"<168598390273.2889415.9003212394211683684@Monstersaurus>","date":"2023-06-05T16:51:42","subject":"Re: [libcamera-devel] [PATCH v2] libcamera: controls: Drop\n\treference to IPA in controls documentation","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"Quoting Laurent Pinchart via libcamera-devel (2023-06-04 09:55:08)\n> The controls relates to the public API, which doesn't expose IPA modules\n> as they are internal to libcamera. Replace the mention of \"IPA\" in the\n> controls documentation.\n> \n> While at it, fix a small text reflow issue.\n> \n> Signed-off-by: Laurent Pinchart \n> Reviewed-by: Naushir Patuck \n> ---\n> Changes since v1:\n> \n> - Drop IPA reference from FocusFoM documentation (thanks Naush)\n> ---\n> src/libcamera/control_ids.yaml | 18 +++++++++---------\n> 1 file changed, 9 insertions(+), 9 deletions(-)\n> \n> diff --git a/src/libcamera/control_ids.yaml b/src/libcamera/control_ids.yaml\n> index adea5f90acc5..a0fbb772de64 100644\n> --- a/src/libcamera/control_ids.yaml\n> +++ b/src/libcamera/control_ids.yaml\n> @@ -275,12 +275,12 @@ controls:\n> type: int32_t\n> description: |\n> Reports a Figure of Merit (FoM) to indicate how in-focus the frame is.\n> - A larger FocusFoM value indicates a more in-focus frame. This control\n> - depends on the IPA to gather ISP statistics from the defined focus\n> - region, and combine them in a suitable way to generate a FocusFoM value.\n> - In this respect, it is not necessarily aimed at providing a way to\n> - implement a focus algorithm by the application, rather an indication of\n> - how in-focus a frame is.\n> + A larger FocusFoM value indicates a more in-focus frame. This singular\n> + value may be based on a combination of statistics gathered from\n> + multiple focus regions within an image. The number of focus regions and\n> + method of combination is platform dependent. In this respect, it is not\n> + necessarily aimed at providing a way to implement a focus algorithm by\n> + the application, rather an indication of how in-focus a frame is.\n\nI wonder if in the future we could adapt a scaling factor to be more\nuseful to applications so that the closer the value is to (an arbitrary\nvalue?) the more in focus it is ... but as we don't explicitly define\nthe values here now - I think this is fine.\n\n\n> \n> - ColourCorrectionMatrix:\n> type: float\n> @@ -333,8 +333,8 @@ controls:\n> - FrameDurationLimits:\n> type: int64_t\n> description: |\n> - The minimum and maximum (in that order) frame duration,\n> - expressed in microseconds.\n> + The minimum and maximum (in that order) frame duration, expressed in\n> + microseconds.\n> \n> When provided by applications, the control specifies the sensor frame\n> duration interval the pipeline has to use. This limits the largest\n> @@ -343,7 +343,7 @@ controls:\n> the sensor will not be able to raise the exposure time above 33ms.\n> A fixed frame duration is achieved by setting the minimum and maximum\n> values to be the same. Setting both values to 0 reverts to using the\n> - IPA provided defaults.\n> + camera defaults.\n\n\nReviewed-by: Kieran Bingham \n\n> \n> The maximum frame duration provides the absolute limit to the shutter\n> speed computed by the AE algorithm and it overrides any exposure mode\n> -- \n> Regards,\n> \n> Laurent Pinchart\n>","headers":{"Return-Path":"","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 532C8C31E9\n\tfor ;\n\tMon, 5 Jun 2023 16:51:48 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 9BA0D62880;\n\tMon, 5 Jun 2023 18:51:47 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id AED036287D\n\tfor ;\n\tMon, 5 Jun 2023 18:51:45 +0200 (CEST)","from pendragon.ideasonboard.com\n\t(aztw-30-b2-v4wan-166917-cust845.vm26.cable.virginm.net\n\t[82.37.23.78])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id B57B0E4;\n\tMon, 5 Jun 2023 18:51:20 +0200 (CEST)"],"DKIM-Signature":["v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org;\n\ts=mail; t=1685983907;\n\tbh=felvJU3DpmoBCE3pPzUXkdYv03wGina8cPH8ih6pho0=;\n\th=In-Reply-To:References:To:Date:Subject:List-Id:List-Unsubscribe:\n\tList-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:\n\tFrom;\n\tb=KWwxJ6wpfhFkplCKqr6XGtF8iQ/yS2dmPWkRn9wijT/7Z3d4dqzr7t1zfVjzF7gmf\n\tvvrdhP/jaX/IwE5lX1DF3bkmi95kTpAnkPMHrKPwgAbvoC7fS05GqwewK9pgR3OzUR\n\tvkeBEZ6oeKTYAajJq9vO4gG+uOvHkvL1/nGDhAfSuRR/kh5alTYn1gnj/9odsnZCAm\n\tTvTD5h9+/JUDO2b5vFJ91Dr5lgt6EYxC90xzMvHCpMnXjDpmnLyUECDop22HD2kFCh\n\tRtl/jM5vrFox4LJyXFr6QLNiTZsWMm8p3m6hBanUJD6sVale5DdNYnfVjZSqb1zj7R\n\tILMplXk+WiYYw==","v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1685983880;\n\tbh=felvJU3DpmoBCE3pPzUXkdYv03wGina8cPH8ih6pho0=;\n\th=In-Reply-To:References:Subject:From:To:Date:From;\n\tb=QZUNp71Ich8VRldHCRzOH0K9ndQ3jDnbERNtvBcWZdzusb3FcMHOLf/X7FbzR3OvF\n\tV3TZQNWUC+C3Kf5dsx/jm913k9XfYf8MaxrcJcYY9lni3Nasg6c52o1gM4AyO7DziF\n\tt5uIAqaeQnPT6TnpE/B5ScXAC6/4/oZ8GVkilPFI="],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key; \n\tunprotected) header.d=ideasonboard.com\n\theader.i=@ideasonboard.com\n\theader.b=\"QZUNp71I\"; dkim-atps=neutral","Content-Type":"text/plain; charset=\"utf-8\"","MIME-Version":"1.0","Content-Transfer-Encoding":"quoted-printable","In-Reply-To":"<20230604085508.1751-1-laurent.pinchart@ideasonboard.com>","References":"<20230604085508.1751-1-laurent.pinchart@ideasonboard.com>","To":"Laurent Pinchart ,\n\tlibcamera-devel@lists.libcamera.org","Date":"Mon, 05 Jun 2023 17:51:42 +0100","Message-ID":"<168598390273.2889415.9003212394211683684@Monstersaurus>","User-Agent":"alot/0.10","Subject":"Re: [libcamera-devel] [PATCH v2] libcamera: controls: Drop\n\treference to IPA in controls documentation","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","From":"Kieran Bingham via libcamera-devel\n\t","Reply-To":"Kieran Bingham ","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}}]