[PATCH] utils: tuning: readme: Improve names of dependency packages
Dan Scally
dan.scally at ideasonboard.com
Mon Mar 18 13:34:26 CET 2024
Hello both
On 18/03/2024 10:47, Laurent Pinchart wrote:
> 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 ?
I was just coming to propose that too. But correcting the package names is still good:
Reviewed-by: Daniel Scally <dan.scally at ideasonboard.com>
>
>> - numpy
>> -- pyexiv2
>> +- py3exiv2
>> - rawpy
More information about the libcamera-devel
mailing list