[libcamera-devel] [PATCH 1/3] utils: raspberrypi: ctt: Fix namespace for sklearn NearestCentroid function

Kieran Bingham kieran.bingham at ideasonboard.com
Mon Jul 26 12:31:55 CEST 2021


Hi David,

On 21/07/2021 12:52, David Plowman wrote:
> The NearestCentroid function is now in the sklearn.neighbors
> namespace.
> 

Does this have requirements of a specific version of the external
import? It would probably be helpful to mention the version that it
became required or ensure that the newest version is now a required
version somehow.

But otherwise,

Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>

> Signed-off-by: David Plowman <david.plowman at raspberrypi.com>
> ---
>  utils/raspberrypi/ctt/ctt_tools.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/utils/raspberrypi/ctt/ctt_tools.py b/utils/raspberrypi/ctt/ctt_tools.py
> index 48e0aac2..8728ff16 100644
> --- a/utils/raspberrypi/ctt/ctt_tools.py
> +++ b/utils/raspberrypi/ctt/ctt_tools.py
> @@ -14,7 +14,7 @@ import imutils
>  import sys
>  import matplotlib.pyplot as plt
>  from sklearn import cluster as cluster
> -from sklearn.neighbors.nearest_centroid import NearestCentroid as get_centroids
> +from sklearn.neighbors import NearestCentroid as get_centroids
>  
>  """
>  This file contains some useful tools, the details of which aren't important to
> 


More information about the libcamera-devel mailing list