<div dir="ltr"><div dir="ltr">Hi Jacopo.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 20 May 2022 at 09:43, Jacopo Mondi <<a href="mailto:jacopo@jmondi.org">jacopo@jmondi.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
On Thu, Apr 21, 2022 at 04:11:16PM +0100, Naushir Patuck via libcamera-devel wrote:<br>
> The SensorSensitivity property is a scaling factor that describes how sensitive<br>
> the selected sensor mode is compared to other readout modes of the same sensor.<br>
> For example, a binned mode might have twice the sensitivity of the full<br>
> resolution mode, meaning you would get double the signal level for the same<br>
> exposure and gains.<br>
><br>
> Signed-off-by: David Plowman <<a href="mailto:david.plowman@raspberrypi.com" target="_blank">david.plowman@raspberrypi.com</a>><br>
> Signed-off-by: Naushir Patuck <<a href="mailto:naush@raspberrypi.com" target="_blank">naush@raspberrypi.com</a>><br>
> ---<br>
>  src/libcamera/property_ids.yaml | 12 ++++++++++++<br>
>  1 file changed, 12 insertions(+)<br>
><br>
> diff --git a/src/libcamera/property_ids.yaml b/src/libcamera/property_ids.yaml<br>
> index 12ecbce5eed4..11b7ebdc3105 100644<br>
> --- a/src/libcamera/property_ids.yaml<br>
> +++ b/src/libcamera/property_ids.yaml<br>
> @@ -678,6 +678,18 @@ controls:<br>
>          \todo Turn this property into a "maximum control value" for the<br>
>          ScalerCrop control once "dynamic" controls have been implemented.<br>
><br>
> +  - SensorSensitivity:<br>
> +      type: float<br>
> +      description: |<br>
> +        The relative sensitivity of the chosen sensor mode.<br>
> +<br>
> +        Some sensors have readout modes with different sensitivities. For example,<br>
> +        a binned camera mode might, with the same exposure and gains, produce<br>
> +        twice the signal level of the full resolution readout. This would be<br>
> +        signalled by the binned mode, when it is chosen, indicating a value here<br>
> +        that is twice that of the full resolution mode. This value will be valid<br>
> +        after the configure method has returned successfully.<br>
<br>
Sorry, I should have payed much more attention during the review.<br>
<br>
The sensitivity depends on the mode applied to the sensor, and as the<br>
description suggests changes at Camera::configure() time.<br>
<br>
Libcamera properties are immutable and they should not depend on the<br>
current configuration but they should stay constant during the whole<br>
lieftime of the Camera.<br>
<br>
This should rather be a Control, even if I understand it cannot be<br>
changed by the user. Do we maybe need to reconsider Properties'<br>
immutability (Laurent in cc for opinions)<br></blockquote><div><br></div><div>ScalerCropMaximum is an existing mutable property as well.  Like</div><div>SensorSensitivity, it is mutable in the sense it is mode specific, but<br></div><div>does not change after a Configure() call.</div><div><br></div><div>Regards,</div><div>Naush</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Thanks<br>
   j<br>
<br>
> +<br>
>    # ----------------------------------------------------------------------------<br>
>    # Draft properties section<br>
><br>
> --<br>
> 2.25.1<br>
><br>
</blockquote></div></div>