Skip to content

test: skip polygon NLPModels test on Julia 1.10 - #331

Merged
adrhill merged 1 commit into
mainfrom
ah/fix-nlp-ci
Oct 7, 2026
Merged

adrhill merged 1 commit into
mainfrom
ah/fix-nlp-ci

Conversation

@adrhill

@adrhill adrhill commented Oct 7, 2026

Copy link
Copy Markdown
Collaborator

The Julia lts (NLPModels) job failed on #330 with:

MethodError: no method matching typemax(::Type{Irrational{:π}})
  [2] _is_typemax(x::Irrational{:π})  @ JuMP variables.jl:146
 [14] polygon(; n)                    @ OptimizationProblems PureJuMP/polygon.jl:21

OptimizationProblems' JuMP model of polygon uses π as a variable bound, and JuMP calls typemax on it. typemax(::Irrational) errors on Julia 1.10 but works on 1.11+. Both jobs resolved the same JuMP (v1.32.0) and OptimizationProblems (v0.9.3), so only the Julia version differs. The error comes from building the JuMP reference model, not from SCT.

This PR:

🤖 Generated with Claude Code

OptimizationProblems' JuMP model of `polygon` uses `π` as a variable
bound, on which JuMP calls `typemax`. That method doesn't exist for
`Irrational` on Julia 1.10, so the LTS NLPModels job errors.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@adrhill

adrhill commented Oct 7, 2026

Copy link
Copy Markdown
Collaborator Author

The two extra invalidations look unrelated.

@adrhill adrhill changed the title test: skip polygon NLPModels test on Julia < 1.11 test: skip polygon NLPModels test on Julia 1.10 Oct 7, 2026
@adrhill
adrhill merged commit e37f4b9 into main Oct 7, 2026
9 of 11 checks passed
@adrhill
adrhill deleted the ah/fix-nlp-ci branch October 7, 2026 15:20
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