FIX add argument to _check_sample_weight (#73)

Fix the following bug:
```
>>> from treeple.ensemble import ObliqueRandomForestClassifier
>>> from sklearn.datasets import make_classification
>>> X,y = make_classification(n_samples=1000, n_features=4,n_informative=2,n_redundant=0,random_state=0,shuffle=False)
>>> clf = ObliqueRandomForestClassifier(max_depth=2, random_state=0)
>>> clf.fit(X,y)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/pssf23/miniconda3/envs/temp/lib/python3.10/site-packages/sklearn/base.py", line 1363, in wrapper
    return fit_method(estimator, *args, **kwargs)
  File "/Users/pssf23/miniconda3/envs/temp/lib/python3.10/site-packages/treeple/_lib/sklearn/ensemble/_forest.py", line 597, in fit
    self._construct_trees(
  File "/Users/pssf23/miniconda3/envs/temp/lib/python3.10/site-packages/treeple/_lib/sklearn/ensemble/_forest.py", line 663, in _construct_trees
    trees = Parallel(
  File "/Users/pssf23/miniconda3/envs/temp/lib/python3.10/site-packages/sklearn/utils/parallel.py", line 82, in __call__
    return super().__call__(iterable_with_config_and_warning_filters)
  File "/Users/pssf23/miniconda3/envs/temp/lib/python3.10/site-packages/joblib/parallel.py", line 1986, in __call__
    return output if self.return_generator else list(output)
  File "/Users/pssf23/miniconda3/envs/temp/lib/python3.10/site-packages/joblib/parallel.py", line 1914, in _get_sequential_output
    res = func(*args, **kwargs)
  File "/Users/pssf23/miniconda3/envs/temp/lib/python3.10/site-packages/sklearn/utils/parallel.py", line 147, in __call__
    return self.function(*args, **kwargs)
  File "/Users/pssf23/miniconda3/envs/temp/lib/python3.10/site-packages/treeple/_lib/sklearn/ensemble/_forest.py", line 211, in _parallel_build_trees
    tree._fit(
  File "/Users/pssf23/miniconda3/envs/temp/lib/python3.10/site-packages/treeple/_lib/sklearn/tree/_classes.py", line 395, in _fit
    sample_weight = _check_sample_weight(sample_weight, X, DOUBLE)
TypeError: _check_sample_weight() takes 2 positional arguments but 3 were given
```

Author: PSSF23

build_tools/circle/build_doc.sh

@@ -176,7 +176,8 @@ conda activate $CONDA_ENV_NAME
 
 show_installed_libraries
 
-pip install -e . --no-build-isolation
+# reduce memory pressure
+pip install scikit-learn
 
 echo "ccache build summary:"
 ccache -s

sklearn/cluster/_hdbscan/_tree.pyx

@@ -783,7 +783,7 @@ cdef tuple _get_clusters(
             else:
                 is_cluster[c] = False
 
-    clusters = set([c for c in is_cluster if is_cluster[c]])
+    clusters = {c for c in is_cluster if is_cluster[c]}
     cluster_map = {c: n for n, c in enumerate(sorted(list(clusters)))}
     reverse_cluster_map = {n: c for c, n in cluster_map.items()}
 

sklearn/ensemble/_forest.py

@@ -1275,7 +1275,7 @@ def partial_fit(self, X, y, sample_weight=None, classes=None):
             multi_output=True,
             accept_sparse="csc",
             dtype=DTYPE,
-            force_all_finite=False,
+            ensure_all_finite=False,
             reset=first_call,
         )
 

sklearn/tree/_classes.py

@@ -401,7 +401,7 @@ def _fit(
         max_leaf_nodes = -1 if self.max_leaf_nodes is None else self.max_leaf_nodes
 
         if sample_weight is not None:
-            sample_weight = _check_sample_weight(sample_weight, X, DOUBLE)
+            sample_weight = _check_sample_weight(sample_weight, X, dtype=DOUBLE)
 
         if y is not None and expanded_class_weight is not None:
             if sample_weight is not None:

sklearn/utils/_cython_blas.pyx

@@ -126,8 +126,8 @@ cdef void _gemv(BLAS_Order order, BLAS_Trans ta, int m, int n, floating alpha,
                 floating beta, floating *y, int incy) noexcept nogil:
     """y := alpha * op(A).x + beta * y"""
     cdef char ta_ = ta
-    if order == RowMajor:
-        ta_ = NoTrans if ta == Trans else Trans
+    if order == BLAS_Order.RowMajor:
+        ta_ = BLAS_Trans.NoTrans if ta == BLAS_Trans.Trans else BLAS_Trans.Trans
         if floating is float:
             sgemv(&ta_, &n, &m, &alpha, <float *> A, &lda, <float *> x,
                   &incx, &beta, y, &incy)
@@ -148,8 +148,10 @@ cpdef _gemv_memview(BLAS_Trans ta, floating alpha, const floating[:, :] A,
     cdef:
         int m = A.shape[0]
         int n = A.shape[1]
-        BLAS_Order order = ColMajor if A.strides[0] == A.itemsize else RowMajor
-        int lda = m if order == ColMajor else n
+        BLAS_Order order = (
+            BLAS_Order.ColMajor if A.strides[0] == A.itemsize else BLAS_Order.RowMajor
+        )
+        int lda = m if order == BLAS_Order.ColMajor else n
 
     _gemv(order, ta, m, n, alpha, &A[0, 0], lda, &x[0], 1, beta, &y[0], 1)
 
@@ -158,7 +160,7 @@ cdef void _ger(BLAS_Order order, int m, int n, floating alpha,
                const floating *x, int incx, const floating *y,
                int incy, floating *A, int lda) noexcept nogil:
     """A := alpha * x.y.T + A"""
-    if order == RowMajor:
+    if order == BLAS_Order.RowMajor:
         if floating is float:
             sger(&n, &m, &alpha, <float *> y, &incy, <float *> x, &incx, A, &lda)
         else:
@@ -175,8 +177,10 @@ cpdef _ger_memview(floating alpha, const floating[::1] x,
     cdef:
         int m = A.shape[0]
         int n = A.shape[1]
-        BLAS_Order order = ColMajor if A.strides[0] == A.itemsize else RowMajor
-        int lda = m if order == ColMajor else n
+        BLAS_Order order = (
+            BLAS_Order.ColMajor if A.strides[0] == A.itemsize else BLAS_Order.RowMajor
+        )
+        int lda = m if order == BLAS_Order.ColMajor else n
 
     _ger(order, m, n, alpha, &x[0], 1, &y[0], 1, &A[0, 0], lda)
 
@@ -194,7 +198,7 @@ cdef void _gemm(BLAS_Order order, BLAS_Trans ta, BLAS_Trans tb, int m, int n,
     cdef:
         char ta_ = ta
         char tb_ = tb
-    if order == RowMajor:
+    if order == BLAS_Order.RowMajor:
         if floating is float:
             sgemm(&tb_, &ta_, &n, &m, &k, &alpha, <float*>B,
                   &ldb, <float*>A, &lda, &beta, C, &ldc)
@@ -214,19 +218,21 @@ cpdef _gemm_memview(BLAS_Trans ta, BLAS_Trans tb, floating alpha,
                     const floating[:, :] A, const floating[:, :] B, floating beta,
                     floating[:, :] C):
     cdef:
-        int m = A.shape[0] if ta == NoTrans else A.shape[1]
-        int n = B.shape[1] if tb == NoTrans else B.shape[0]
-        int k = A.shape[1] if ta == NoTrans else A.shape[0]
+        int m = A.shape[0] if ta == BLAS_Trans.NoTrans else A.shape[1]
+        int n = B.shape[1] if tb == BLAS_Trans.NoTrans else B.shape[0]
+        int k = A.shape[1] if ta == BLAS_Trans.NoTrans else A.shape[0]
         int lda, ldb, ldc
-        BLAS_Order order = ColMajor if A.strides[0] == A.itemsize else RowMajor
+        BLAS_Order order = (
+            BLAS_Order.ColMajor if A.strides[0] == A.itemsize else BLAS_Order.RowMajor
+        )
 
-    if order == RowMajor:
-        lda = k if ta == NoTrans else m
-        ldb = n if tb == NoTrans else k
+    if order == BLAS_Order.RowMajor:
+        lda = k if ta == BLAS_Trans.NoTrans else m
+        ldb = n if tb == BLAS_Trans.NoTrans else k
         ldc = n
     else:
-        lda = m if ta == NoTrans else k
-        ldb = k if tb == NoTrans else n
+        lda = m if ta == BLAS_Trans.NoTrans else k
+        ldb = k if tb == BLAS_Trans.NoTrans else n
         ldc = m
 
     _gemm(order, ta, tb, m, n, k, alpha, &A[0, 0],