Skip to content

Commit 1bf8ab2

Browse files
authored
fix: prune installed features in workflow to avoid disk exceed limit (#1296)
* fix: prune installed features in workflow to avoid disk exceed limit * fix: explicitly install maturin
1 parent 9f09062 commit 1bf8ab2

File tree

2 files changed

+7
-15
lines changed

2 files changed

+7
-15
lines changed

.github/workflows/_test.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,10 +64,10 @@ jobs:
6464
python -m venv .venv
6565
- name: Install Python toolchains
6666
run: |
67-
${{ matrix.platform.python_exec }} -m pip install maturin mypy pytest pytest-asyncio
67+
${{ matrix.platform.python_exec }} -m pip install maturin
6868
- name: Python build
6969
run: |
70-
${{ matrix.platform.python_exec }} -m maturin develop --strip -E all
70+
${{ matrix.platform.python_exec }} -m maturin develop --strip -E all-ci
7171
- name: Python type check (mypy)
7272
run: |
7373
${{ matrix.platform.python_exec }} -m mypy python

pyproject.toml

Lines changed: 5 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -69,16 +69,13 @@ dev = ["pytest", "pytest-asyncio", "ruff", "mypy", "pre-commit"]
6969
embeddings = ["sentence-transformers>=3.3.1"]
7070
colpali = ["colpali-engine"]
7171
lancedb = ["lancedb>=0.25.0"]
72-
pydantic = ["pydantic>=2.11.9"]
7372

7473
# We need to repeat the dependency above to make it available for the `all` feature.
7574
# Indirect dependencies such as "cocoindex[embeddings]" will not work for local development.
76-
all = [
77-
"sentence-transformers>=3.3.1",
78-
"colpali-engine",
79-
"lancedb>=0.25.0",
80-
"pydantic>=2.11.9",
81-
]
75+
all = ["sentence-transformers>=3.3.1", "colpali-engine", "lancedb>=0.25.0"]
76+
77+
# This is the set of dependencies that needs to be installed for the CI workflow.
78+
all-ci = ["pytest", "pytest-asyncio", "mypy", "pydantic>=2.11.9"]
8279

8380
[tool.mypy]
8481
python_version = "3.11"
@@ -87,10 +84,5 @@ files = "python/cocoindex"
8784
exclude = "(\\.venv|site-packages)"
8885
disable_error_code = ["unused-ignore"]
8986
[[tool.mypy.overrides]]
90-
module = [
91-
"sentence_transformers",
92-
"torch",
93-
"colpali_engine",
94-
"PIL",
95-
]
87+
module = ["sentence_transformers", "torch", "colpali_engine", "PIL"]
9688
ignore_missing_imports = true

0 commit comments

Comments
 (0)