[PATCH] utils: tuning: readme: Improve names of dependency packages

Laurent Pinchart laurent.pinchart at ideasonboard.com
Mon Mar 18 11:47:36 CET 2024


Hi Paul,

Thank you for the patch.

On Mon, Mar 18, 2024 at 07:13:31PM +0900, Paul Elder wrote:
> Change the names of python packages in the list of dependencies to the
> exact names that should be used in the pip install command.

Sounds like a good idea.

> Signed-off-by: Paul Elder <paul.elder at ideasonboard.com>
> ---
>  utils/tuning/README.rst | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/utils/tuning/README.rst b/utils/tuning/README.rst
> index ce533b2c..ce3e03fc 100644
> --- a/utils/tuning/README.rst
> +++ b/utils/tuning/README.rst
> @@ -5,7 +5,7 @@
>  Dependencies
>  ------------
>  
> -- cv2
> +- opencv-python

With alphabetical order preserved,

Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

I wonder if we should have a requirements.txt file ?

>  - numpy
> -- pyexiv2
> +- py3exiv2
>  - rawpy

-- 
Regards,

Laurent Pinchart


More information about the libcamera-devel mailing list