Skip to content

Allow JET 0.12 in the QA environment for Julia 1.13 - #682

Merged
ChrisRackauckas merged 1 commit into
masterfrom
jet-0.12
Oct 15, 2026
Merged

ChrisRackauckas merged 1 commit into
masterfrom
jet-0.12

Conversation

@ChrisRackauckas

@ChrisRackauckas ChrisRackauckas commented Oct 15, 2026 •

Copy link
Copy Markdown
Member

🚧 UNREVIEWED — awaiting review by @ChrisRackauckas. Filed by an AI agent running as @ChrisRackauckas; Chris has not seen this.
Harness: Devin CLI 3000.11.92 (diagnosis and fix), Claude Code 2.1.300 (PR) · Model: fusion-claude-opus-5-5-high-sidekick-swe-2-medium
Conversation: https://claude.ai/code/session_01VZjwf7Ux9YtTzwHGHMqnsq (fleet master)

This fixes a failure on the default branch's CI. The diagnosis and the evidence below come from the investigating agent's report; "confirmed by running" and "read in logs" are kept apart there.

Change

JET = "0.9, 0.10, 0.11" -> "0.9, 0.10, 0.11, 0.12" in:

  • lib/RecursiveArrayToolsArrayPartitionAnyAll/test/qa/Project.toml
  • lib/RecursiveArrayToolsRaggedArrays/test/qa/Project.toml
  • lib/RecursiveArrayToolsShorthandConstructors/test/qa/Project.toml
  • test/NoPre/Project.toml

No .jl files changed, so Runic was not needed. typos on the changed files: clean.

Environment

export TMPDIR=<sandbox>/tmp CUDA_VISIBLE_DEVICES="" JULIA_NUM_THREADS=4

JET is not in the root QA env (test/QA). It is used by the NoPre group and by each sublibrary's QA env, which runs from lib/<Sublib> with RECURSIVEARRAYTOOLS_TEST_GROUP=QA.

Commands and results

Commands:

  • Root QA: GROUP=QA julia +<ch> --project -e 'using Pkg; Pkg.test()' (repo root)
  • NoPre: GROUP=NoPre julia +<ch> --project -e 'using Pkg; Pkg.test()' (repo root)
  • Sublib QA: RECURSIVEARRAYTOOLS_TEST_GROUP=QA julia +<ch> --project -e 'using Pkg; Pkg.test()' (in lib/)

Before (master, Julia 1.13.2): NoPre and all 3 sublib QA envs fail to resolve:
Unsatisfiable requirements detected for package JET ... restricted by julia compatibility requirements to versions: 0.12.0 - 0.12.3 or uninstalled — no versions left

After (branch):

Run Julia 1.13.2 (+release) Julia 1.10.12 (+lts)
Root QA PASS 20/20 FAIL 17/18 (already fails on master, see below)
NoPre (JET report_package) PASS 1/1, JET 0.12.3 PASS 1/1, JET 0.9.18
RaggedArrays QA PASS 20 + 1 broken, JET 0.12.3 PASS 19/19, JET 0.9.18
ShorthandConstructors QA PASS 21/21, JET 0.12.3 PASS 19/19, JET 0.9.18
ArrayPartitionAnyAll QA PASS 21/21, JET 0.12.3 PASS 19/19, JET 0.9.18

JET reports

JET 0.12.3 reports nothing new, so no code fixes were needed. The one "broken" result in RaggedArrays on 1.13 comes from the jet_broken = VERSION >= v"1.11" lane that was already there (tracked in #620). JET 0.12 still reports that finding, so the result is an expected broken test, not an unexpected pass. On LTS, JET resolves to 0.9.18 both before and after this change.

Pre-existing failure (not addressed)

Root GROUP=QA on Julia 1.10.12 fails on unmodified master with the same output. Aqua finds 1 method ambiguity:
similar(::AbstractVectorOfArray, ::Type{T}, ::Tuple{Union{Integer,OneTo},Vararg{...}}) (src/vector_of_array.jl:1157) vs Base similar(::AbstractArray, ::Type{T}, ::Tuple{Integer,Vararg{Integer}}).
Julia 1.13 does not report it. It needs a separate fix (Aqua suggests defining similar(::AbstractVectorOfArray, ::Type{T}, ::Tuple{Integer,Vararg{Integer}})) in its own PR.

Not run

Nothing was skipped. Only the QA and NoPre groups were run; the other test groups were out of scope.


🤖 Posted by an AI agent — harness: Devin CLI 3000.11.92 (fix), Claude Code 2.1.300 (PR) · model: fusion-claude-opus-5-5-high-sidekick-swe-2-medium / claude-opus-5-5
Conversation: https://claude.ai/code/session_01VZjwf7Ux9YtTzwHGHMqnsq (fleet master)

🤖 Generated with Claude Code

Independent review (added by the fleet master)

Risk assessment

  • Risk: low
  • Blast radius: JET version resolution in lib/RecursiveArrayToolsArrayPartitionAnyAll/test/qa/Project.toml, lib/RecursiveArrayToolsRaggedArrays/test/qa/Project.toml, lib/RecursiveArrayToolsShorthandConstructors/test/qa/Project.toml, test/NoPre/Project.toml; JET 0.12 may expose QA reports. No runtime source or public API changes.
  • Evidence: gh pr diff shows 4 JET compat append(s) in lib/RecursiveArrayToolsArrayPartitionAnyAll/test/qa/Project.toml, lib/RecursiveArrayToolsRaggedArrays/test/qa/Project.toml, lib/RecursiveArrayToolsShorthandConstructors/test/qa/Project.toml, test/NoPre/Project.toml; all existing bounds retained; no other changed lines, source edits, silencing, sources/path edits, or other dependency edits. gh pr checks: QA pending — no QA job reported yet; test discovery/workflows are queued or running. Local tests not run; PR body not read.
  • Independent review: codex gpt-6.1-sol rated it low, high confidence in diff scope: every edit only appends JET 0.12 while retaining existing bounds; runtime compatibility is unverified while QA is pending.
  • Merge: auto-merge candidate — focused compat-only change; require successful QA and required checks before merging

🤖 Section added by an AI agent — harness: Claude Code 2.1.300 · model: claude-opus-5-5 (review by codex-cli, gpt-6.1-sol)
Conversation: https://claude.ai/code/session_01VZjwf7Ux9YtTzwHGHMqnsq

General now restricts JET 0.10.15 and 0.11.4-0.11.6 to Julia 1.12, so the
JET-using test environments cannot resolve on Julia 1.13 unless JET 0.12
is allowed.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Devin <noreply@cognition.ai>
Agent-Harness: Devin CLI 3000.11.92
Agent-Model: fusion-claude-opus-5-5-high-sidekick-swe-2-medium
Agent-Session: local headless Devin run (fleet master JET lane; no session URL)
@ChrisRackauckas
ChrisRackauckas marked this pull request as ready for review October 15, 2026 22:50
@ChrisRackauckas
ChrisRackauckas merged commit 76bec79 into master Oct 15, 2026
69 of 71 checks passed
@ChrisRackauckas
ChrisRackauckas deleted the jet-0.12 branch October 15, 2026 22:50
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.

1 participant