1 package khisto python by bruno-at-orange · Pull Request #5 · KhiopsML/khisto-python

bruno-at-orange · 2026-02-18T15:46:01Z

Add pip packaging with cmake

We use scikit-build-core, a build backend for Python that uses CMake
Khiops sources are download with the fetch mechanism of CMake
The version of khiops is specify in pyproject.toml
CMake extract the version from the file pyproject.toml
The version is specified in the "tool.khiops" section as "khiops-version = "x.y.z""

Add copyright in headers of source files: close #4

pyproject.toml

It fails with the errors: - hook id: pyrefly - exit code: 1 INFO Found `/Users/bruno/dvpt/khisto-python/pyproject.toml` marking project root, checking root directory with default configuration ERROR Cannot set item in `dict[str | Any, list[float]]` [unsupported-operation] --> src/khisto/core/cli_adapter.py:141:27 | 141 | result[col] = np.array([float(row[col]) for row in rows], dtype=np.float64) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | Argument `ndarray[tuple[Any, ...], dtype[float64]]` is not assignable to parameter `value` with type `list[float]` in function `dict.__setitem__` ERROR Cannot set item in `dict[str | Any, list[float]]` [unsupported-operation] --> src/khisto/core/cli_adapter.py:143:27 | 143 | result[col] = np.array([int(row[col]) for row in rows], dtype=np.int64) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | Argument `ndarray[tuple[Any, ...], dtype[signedinteger[_64Bit]]]` is not assignable to parameter `value` with type `list[float]` in function `dict.__setitem__` ERROR Cannot set item in `dict[str | Any, list[float]]` [unsupported-operation] --> src/khisto/core/cli_adapter.py:146:27 | 146 | result[col] = np.array([row[col] for row in rows], dtype=object) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | Argument `ndarray[tuple[Any, ...], dtype[Any]]` is not assignable to parameter `value` with type `list[float]` in function `dict.__setitem__` ERROR Returned type `dict[str | Any, list[float]]` is not assignable to declared return type `dict[str, ndarray[tuple[Any, ...], dtype[Any]]]` [bad-return] --> src/khisto/core/cli_adapter.py:148:12 | 148 | return result | ^^^^^^ | INFO 4 errors

.github/workflows/pack-pip.yml

.pre-commit-config.yaml

CMakeLists.txt

pyproject.toml

popescu-v

Several comments (most of them just questions).

- We use scikit-build-core, a build backend for Python that uses CMake - Khiops sources are download with the fetch mechanism of CMake - The version of khiops is specify in pyproject.toml CMake extract the version from the file pyproject.toml The version is specified in the "tool.khiops" section as "khiops-version = "x.y.z""

popescu-v

One CI comment pending.

pyproject.toml

.github/workflows/pack-pip.yml

pyproject.toml

popescu-v

Just two pending questions (see the comments).

.github/workflows/pack-pip.yml

popescu-v · 2026-02-27T10:23:42Z

.github/workflows/pack-pip.yml

+      - name: Test built wheel
+        run: |
+          pip install --find-links=wheelhouse khisto
+          pip install pytest>=8.3 pytest-cov>=6 pytest-xdist>=3.6 pytest-sugar>=1.0


I'd put these test dependencies into a test-requirements.txt file, then just do pip install -r test-requirements.txt.

scripts/check-obsolete-copyright.py

scripts/update-copyright.py

CMakeLists.txt

bruno-at-orange linked an issue Feb 18, 2026 that may be closed by this pull request

Package khisto-python #1

Closed

bruno-at-orange force-pushed the 1-package-khisto-python branch from 09b2d2f to 886fe05 Compare February 18, 2026 17:50

bruno-at-orange commented Feb 19, 2026

View reviewed changes

pyproject.toml Outdated Show resolved Hide resolved

bruno-at-orange added 3 commits February 20, 2026 10:00

chore: remove git submodule for khiops sources

60e29ce

chore: add license file

bb9d392

bruno-at-orange force-pushed the 1-package-khisto-python branch from 886fe05 to 7031528 Compare February 20, 2026 09:16

bruno-at-orange marked this pull request as ready for review February 20, 2026 09:17

bruno-at-orange requested a review from popescu-v February 20, 2026 09:17