fix: Bump stale compat floors for EvoTrees, OffsetArrays, StatsBase#122
Open
sethaxen wants to merge 2 commits into
Open
fix: Bump stale compat floors for EvoTrees, OffsetArrays, StatsBase#122sethaxen wants to merge 2 commits into
sethaxen wants to merge 2 commits into
Conversation
These floors were unreachable together and broke the Downgrade CI job's
merged-resolution step ("Unsatisfiable"):
- StatsBase 0.33.7 -> 0.33.17: required by PosteriorStats>=0.3
- OffsetArrays 1 -> 1.0.1: Interpolations (via PosteriorStats -> KernelDensity)
excludes exactly 1.0.0
- EvoTrees 0.16 -> 0.16.3: CUDA was a hard dep before 0.16.3, and that CUDA
version's GPUCompiler requirement doesn't support Julia 1.10
Verified by resolving and running the full test suite with all deps pinned
to these floor versions on Julia 1.10.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
StatsBase is a weakdep, and its compat floor changed in the previous commit. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #122 +/- ##
==========================================
+ Coverage 91.58% 92.16% +0.57%
==========================================
Files 24 24
Lines 535 536 +1
==========================================
+ Hits 490 494 +4
+ Misses 45 42 -3 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The Downgrade CI job (
Julia min) has been failing onmainwithERROR: LoadError: Unsatisfiableduring the merged-resolution step. Reproduced locally withPkg.resolve()(not just Resolver.jl's terse error) and traced it to three genuinely-unreachable compat floors, all confirmed against the General registry:StatsBase = "0.33.7, 0.34"→"0.33.17, 0.34":PosteriorStats(weakdep, floor0.3) requiresStatsBase 0.33.17-0.34starting atPosteriorStatsv0.3.0, so our claimed0.33.7floor was never actually reachable alongside our ownPosteriorStatsfloor.OffsetArrays = "1"→"1.0.1": resolves to exactly1.0.0, butInterpolations(pulled in transitively viaPosteriorStats → KernelDensity → Interpolations) explicitly excludes1.0.0(compat: [0.10-0.11, 1.0.1-1]).EvoTrees = "0.16"→"0.16.3": resolves to0.16.0, whereCUDAwas still a hard dependency (not a weakdep), and thatCUDAversion'sGPUCompilerrequirement doesn't support Julia 1.10 (ourjuliafloor).CUDAbecame a weakdep ofEvoTreesstarting at v0.16.3.Since
StatsBaseis a weakdep (triggersInferenceObjectsPosteriorStatsExt) and its compat floor changed, this also bumps the patch version per repo convention.Verification
Pkg.resolve()on the merged (extras-promoted) project now succeeds on Julia 1.10, where it previously errored withUnsatisfiable.[compat]pinned to the new floor versions on the realProject.toml(weakdeps/extensions intact),Pkg.test()passes cleanly: 2085 passed, 21 broken (pre-existing), 0 failed, 0 errored. All three packages resolve at exactly their new floors, and all three extensions (NCDatasets,MCMCDiagnosticTools,PosteriorStats) load correctly.Pkg.resolve()at latest versions still succeeds (only lower bounds changed).Test plan
Downgrade / Julia min - ubuntu-latestjob🤖 Generated with Claude Code