Skip to content

Fix missing test deps, docs, and downgrade CI#232

Merged
Krastanov merged 1 commit into
qojulia:masterfrom
Krastanov-agent:fix-ci-bloch-tests-codex
Jun 18, 2026
Merged

Fix missing test deps, docs, and downgrade CI#232
Krastanov merged 1 commit into
qojulia:masterfrom
Krastanov-agent:fix-ci-bloch-tests-codex

Conversation

@Krastanov-agent

@Krastanov-agent Krastanov-agent commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Document the exported Bloch sphere plotting APIs so the docs checkdocs=:exports build passes.
  • Add missing test dependencies: Makie, SciMLBase for ODEProblem/solve, and OrdinaryDiffEqLowOrderRK for DP5.
  • Keep the SciML solution access fix to sol.u[end].
  • Keep downgrade CI on allow_reresolve=false and raise the minimum compat floors needed for the downgraded root manifest and no-reresolve test environment.

Downgrade bounds

  • Raise Adapt, FFTW, RecursiveArrayTools, and SpecialFunctions lower bounds to versions that resolve together under julia-downgrade-compat forced deps.
  • Add weak compat floors for MKL_jll, SciMLBase, and SciMLOperators so the test environment does not resolve incompatible SciML/LinearSolve transitives.

Validation

  • Focused SciML TestItemRunner run in a temporary environment: 2 tests passed.
  • Focused plotting TestItemRunner run in a temporary environment: 9 tests passed.
  • JULIA_PKG_SERVER_REGISTRY_PREFERENCE=eager julia --project=docs docs/make.jl
  • JULIA_PKG_SERVER_REGISTRY_PREFERENCE=eager julia +1.10 /tmp/julia-downgrade-compat.jl "Pkg,TOML,InteractiveUtils,Random,LinearAlgebra,SparseArrays" "." "forcedeps" "1.10"
  • JULIA_PKG_SERVER_REGISTRY_PREFERENCE=eager julia +1.10 --project=. -e 'using Pkg; Pkg.build(verbose=true)'
  • JULIA_PKG_SERVER_REGISTRY_PREFERENCE=eager julia +1.10 --project=. -e 'import Pkg; Pkg.test(; coverage=true, julia_args=["--check-bounds=yes", "--compiled-modules=yes", "--depwarn=yes"], test_args=["general"], force_latest_compatible_version=false, allow_reresolve=false)' passed: 2128 pass, 2 broken.

Notes

  • No Buildkite/JET changes remain in this PR.

  • No test/runtests.jl changes remain in this PR.

  • No plotting test rewrite remains in this PR.

  • Repo-wide whitespace check reports pre-existing unrelated whitespace issues; this PR diff is clean under git diff --check.

  • GitHub Downgrade test (1.10) passed on commit fe245cb.

  • Removed JET from test/Project.toml; test/runtests.jl now installs it with Pkg.add("JET") only when JET_TEST=true, matching the conditional GPU dependency pattern.

  • Latest validation after the JET change: julia --project=. -e 'using QuantumOpticsBase; @assert isdefined(@__MODULE__, :blochsphereplot_axis); @assert !isdefined(@__MODULE__, :JET)' passed.

  • Latest normal Pkg.test(; test_args=["general"]) on local Julia 1.12 got past JET precompilation and then failed in test/test_plotting.jl with blochsphereplot_axis not in scope inside the test item; this is separate from the JET dependency change.

@Krastanov-agent Krastanov-agent force-pushed the fix-ci-bloch-tests-codex branch 2 times, most recently from 8a882bb to 0586091 Compare June 11, 2026 19:37
@Krastanov-agent Krastanov-agent changed the title Fix CI test groups and docs Fix missing test dependencies and docs Jun 11, 2026
@Krastanov-agent Krastanov-agent force-pushed the fix-ci-bloch-tests-codex branch from 0586091 to 212fc85 Compare June 11, 2026 19:55
@Krastanov-agent Krastanov-agent changed the title Fix missing test dependencies and docs Fix missing test deps, docs, and downgrade CI Jun 11, 2026
@Krastanov-agent Krastanov-agent force-pushed the fix-ci-bloch-tests-codex branch from 212fc85 to fe245cb Compare June 11, 2026 21:04
@Krastanov-agent Krastanov-agent force-pushed the fix-ci-bloch-tests-codex branch from fe245cb to bf81763 Compare June 18, 2026 05:20
@Krastanov Krastanov merged commit 9e713e2 into qojulia:master Jun 18, 2026
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants