-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Install lowest direct in CI #3765
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
e4bd805
0655a1f
62dd096
20f5ff2
c2b437e
9a9ba43
3b12576
b228728
d06e41a
e42d1dd
364d5b4
a1594af
1c0aad9
429bbde
109ac3b
592786d
49794dc
7d4c6bc
d5f9cfa
a7f5736
7a90bfe
5958524
133a98a
c4f1eeb
48b6562
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -169,9 +169,8 @@ jobs: | |
|
|
||
| - run: mkdir .coverage | ||
|
|
||
| - run: uv sync --only-dev | ||
|
|
||
| - run: uv run mcp-run-python example --deps=numpy | ||
| # We install the `--group dev` first because if we use `--package` together, it will try to install the `dev` group from the package. | ||
| - run: uv sync --group dev | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Adding |
||
|
|
||
| - name: cache HuggingFace models | ||
| uses: actions/cache@v4 | ||
|
|
@@ -195,16 +194,16 @@ jobs: | |
| test-lowest-versions: | ||
| name: test on ${{ matrix.python-version }} (lowest-versions) | ||
| runs-on: ubuntu-latest | ||
| timeout-minutes: 20 | ||
| timeout-minutes: 35 | ||
| strategy: | ||
| fail-fast: false | ||
| matrix: | ||
| # TODO(Marcelo): Enable 3.11 again. | ||
| python-version: ["3.10", "3.12", "3.13"] | ||
| python-version: ["3.10", "3.11", "3.12", "3.13"] | ||
| env: | ||
| CI: true | ||
| COVERAGE_PROCESS_START: ./pyproject.toml | ||
| RUN_LLAMA_CPP_TESTS: false | ||
| UV_FROZEN: "0" | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
|
|
||
|
|
@@ -220,9 +219,7 @@ jobs: | |
|
|
||
| - run: mkdir .coverage | ||
|
|
||
| - run: uv sync --group dev | ||
|
|
||
| - run: uv run mcp-run-python example --deps=numpy | ||
| - run: uv sync --group dev --resolution lowest-direct --all-extras | ||
|
|
||
| - name: cache HuggingFace models | ||
| uses: actions/cache@v4 | ||
|
|
@@ -232,9 +229,7 @@ jobs: | |
| restore-keys: | | ||
| hf-${{ runner.os }}- | ||
|
|
||
| - run: unset UV_FROZEN | ||
|
|
||
| - run: uv run --all-extras --resolution lowest-direct coverage run -m pytest --durations=100 -n auto --dist=loadgroup | ||
| - run: uv run --no-sync coverage run -m pytest --durations=100 -n auto --dist=loadgroup | ||
| env: | ||
| COVERAGE_FILE: .coverage/.coverage.${{matrix.python-version}}-lowest-versions | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -57,7 +57,7 @@ dependencies = [ | |
| "httpx>=0.27", | ||
| "pydantic>=2.10", | ||
| "pydantic-graph=={{ version }}", | ||
| "exceptiongroup; python_version < '3.11'", | ||
| "exceptiongroup>=1.2.2; python_version < '3.11'", | ||
| "opentelemetry-api>=1.28.0", | ||
| "typing-inspection>=0.4.0", | ||
| "genai-prices>=0.0.40", | ||
|
|
@@ -77,11 +77,11 @@ openrouter = ["openai>=2.8.0"] | |
| mistral = ["mistralai>=1.9.10"] | ||
| bedrock = ["boto3>=1.40.14"] | ||
| huggingface = ["huggingface-hub[inference]>=0.33.5,<1.0.0"] | ||
| outlines-transformers = ["outlines[transformers]>=1.0.0, <1.3.0; (sys_platform != 'darwin' or platform_machine != 'x86_64')", "transformers>=4.0.0", "pillow", "torch; (sys_platform != 'darwin' or platform_machine != 'x86_64')"] | ||
| outlines-transformers = ["outlines[transformers]>=1.0.0, <1.3.0; (sys_platform != 'darwin' or platform_machine != 'x86_64')", "transformers>=4.0.0", "pillow>11.0.0", "torch>=2.8.0; (sys_platform != 'darwin' or platform_machine != 'x86_64')"] | ||
| outlines-llamacpp = ["outlines[llamacpp]>=1.0.0, <1.3.0"] | ||
| outlines-mlxlm = ["outlines[mlxlm]>=1.0.0, <1.3.0; platform_system == 'Darwin' and platform_machine == 'arm64'"] | ||
| outlines-sglang = ["outlines[sglang]>=1.0.0, <1.3.0", "pillow"] | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Bug: Inconsistent version specifier excludes Pillow 11.0.0The |
||
| outlines-vllm-offline = ["vllm; python_version < '3.12' and (sys_platform != 'darwin' or platform_machine != 'x86_64')", "torch; (sys_platform != 'darwin' or platform_machine != 'x86_64')", "outlines>=1.0.0, <1.3.0"] | ||
| outlines-sglang = ["outlines[sglang]>=1.0.0, <1.3.0", "pillow>11.0.0"] | ||
| outlines-vllm-offline = ["vllm>=0.8.0; python_version < '3.12' and (sys_platform != 'darwin' or platform_machine != 'x86_64')", "torch>=2.8.0; (sys_platform != 'darwin' or platform_machine != 'x86_64')", "outlines>=1.0.0, <1.3.0"] | ||
| # Tools | ||
| duckduckgo = ["ddgs>=9.0.0"] | ||
| tavily = ["tavily-python>=0.5.0"] | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DouweM is this still needed now that we have the
mcp-run-pythonin another repo? I've already checked that this test is also ran on that repo.Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Kludex I'm fine removing it.
Note that we also test that it
mcp-run-pythonbe started here:pydantic-ai/docs/mcp/client.md
Lines 140 to 152 in c1213e4
That makes the test suite require deno. I actually think we should drop that entirely and really treat it as a separate package.