fix(deps): resolve gpu/benchmarks extras conflict for uv - #917
Conversation
cuopt-cu12 (gpu) pulls cudf-cu12 which pins pandas<3.0.4, clashing with the benchmarks pandas==3.0.5 pin and making uv resolution unsatisfiable. Declare the two extras conflicting so uv resolves them in separate forks.
Build cost — v1 vs legacyv1 build peak & time relative to legacy, on this commit — not a comparison against master (that is CodSpeed).
Full table (time + peak, mean)📊 Interactive plots + CSV: download the semantics-report-v1-vs-legacy artifact from this run. Report-only · not a gate · refreshed on every push · obsolete once legacy is dropped. |
Merging this PR will degrade performance by 6.24%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Memory | test_to_lp[masked-n=100] |
1.5 MB | 2.7 MB | -45.22% |
| ❌ | Memory | test_to_lp[rolling-severity=0] |
2.1 MB | 2.9 MB | -28.11% |
| ⚡ | Memory | test_to_lp[knapsack-n=10000] |
2.7 MB | 1.9 MB | +45.48% |
| ⚡ | Memory | test_to_lp[rolling-severity=50] |
429.7 MB | 318.7 MB | +34.85% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing fix/uv-extras-conflict (d107c4f) with master (6b69f96)
Footnotes
-
175 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩
Since #909,
uv lock/uv syncfail for everyone on master. This unblocks the dev environment again.Note
The following content was generated by AI.
Problem
#909addedcuopt-cu12>=26.8to thegpuextra. It pullscudf-cu12, which pinspandas>=3.0.0,<3.0.4a0. Thebenchmarksextra pinspandas==3.0.5. The two are mutually incompatible, and the dev resolution requires both, souvresolution is unsatisfiable:Fix
Declare the two extras as conflicting so
uvresolves them in separate forks instead of unifying them:After this,
uv lockresolves (263 packages),uv lock --checkpasses, anduv sync --extra devworks.gpuandbenchmarksremain individually installable but not together, which reflects the underlying pandas incompatibility.Checklist