Skip to content

[CI] Fixed lowest Gridap version parsing - #244

Merged
miguelmaso merged 1 commit into
mainfrom
version-parse
Oct 20, 2026
Merged

miguelmaso merged 1 commit into
mainfrom
version-parse

Conversation

@miguelmaso

@miguelmaso miguelmaso commented Oct 19, 2026 •

Copy link
Copy Markdown
Collaborator

#240 added a new workflow to check the backward compatibility of the lowest Gridap version allowed by Project.toml.

The first implementation only strips the ^ and ~ before the first version number. This PR moves to the Pkg own parser, handling ^, ~, =, ≥, <, hyphen ranges and comma-separated unions.

@github-actions

Copy link
Copy Markdown

Benchmark Results (Julia v1.13.1)

Time benchmarks
main 801e54b... main / 801e54b...
Constitutive models/Visco-elastic Ψ 8.58 ± 6.4 μs 9.85 ± 5.7 μs 0.871 ± 0.83
Constitutive models/Visco-elastic ∂Ψ∂F 18.3 ± 6.3 μs 16.8 ± 6.1 μs 1.09 ± 0.54
Constitutive models/Visco-elastic ∂∂Ψ∂FF 24.4 ± 13 μs 23.2 ± 12 μs 1.05 ± 0.77
Constitutive models/Visco-polyconvex Ψ 0.13 ± 0 μs 0.12 ± 0.01 μs 1.08 ± 0.09
Constitutive models/Visco-polyconvex ∂Ψ∂F 0.14 ± 0 μs 0.14 ± 0 μs 1 ± 0
Constitutive models/Visco-polyconvex ∂∂Ψ∂FF 0.571 ± 0.01 μs 0.541 ± 0.001 μs 1.06 ± 0.019
Simulations/StaticMechanicalDirichlet 0.104 ± 0.005 s 0.105 ± 0.0059 s 0.994 ± 0.074
Simulations/StaticMechanicalNeumann 0.0864 ± 0.01 s 0.084 ± 0.01 s 1.03 ± 0.18
Simulations/ViscoElastic 0.618 ± 0.0068 s 0.61 ± 0.021 s 1.01 ± 0.036
Tensor algebra/Cofactor 20 ± 10 ns 30 ± 10 ns 0.667 ± 0.4
Tensor algebra/Det(A)Inv(A') 30 ± 10 ns 0.04 ± 0.01 μs 0.75 ± 0.31
Tensor algebra/IIsym 30 ± 0 ns 30 ± 10 ns 1 ± 0.33
Tensor algebra/push_forward_C_to_F 0.12 ± 0.01 μs 0.12 ± 0.009 μs 1 ± 0.11
Tensor algebra/×ᵢ⁴ 20 ± 10 ns 20 ± 10 ns 1 ± 0.71
Tensor algebra/δδ_λ_2d 20 ± 0 ns 20 ± 0 ns 1 ± 0
Tensor algebra/δδ_μ_2d 20 ± 0 ns 20 ± 0 ns 1 ± 0
time_to_load 1.57 ± 0.041 s 1.62 ± 0.011 s 0.97 ± 0.026
Memory benchmarks
main 801e54b... main / 801e54b...
Constitutive models/Visco-elastic Ψ 0.186 k allocs: 25.6 kB 0.186 k allocs: 25.6 kB 1
Constitutive models/Visco-elastic ∂Ψ∂F 0.235 k allocs: 30.5 kB 0.235 k allocs: 30.5 kB 1
Constitutive models/Visco-elastic ∂∂Ψ∂FF 0.482 k allocs: 0.0562 MB 0.482 k allocs: 0.0562 MB 1
Constitutive models/Visco-polyconvex Ψ 0 allocs: 0 B 0 allocs: 0 B
Constitutive models/Visco-polyconvex ∂Ψ∂F 0 allocs: 0 B 0 allocs: 0 B
Constitutive models/Visco-polyconvex ∂∂Ψ∂FF 0 allocs: 0 B 0 allocs: 0 B
Simulations/StaticMechanicalDirichlet 1.51 M allocs: 0.108 GB 1.51 M allocs: 0.108 GB 1
Simulations/StaticMechanicalNeumann 1.46 M allocs: 0.0916 GB 1.46 M allocs: 0.0916 GB 1
Simulations/ViscoElastic 12.7 M allocs: 0.979 GB 12.7 M allocs: 0.979 GB 1
Tensor algebra/Cofactor 0 allocs: 0 B 0 allocs: 0 B
Tensor algebra/Det(A)Inv(A') 0 allocs: 0 B 0 allocs: 0 B
Tensor algebra/IIsym 0 allocs: 0 B 0 allocs: 0 B
Tensor algebra/push_forward_C_to_F 0 allocs: 0 B 0 allocs: 0 B
Tensor algebra/×ᵢ⁴ 0 allocs: 0 B 0 allocs: 0 B
Tensor algebra/δδ_λ_2d 0 allocs: 0 B 0 allocs: 0 B
Tensor algebra/δδ_μ_2d 0 allocs: 0 B 0 allocs: 0 B
time_to_load 0.2 k allocs: 11.7 kB 0.2 k allocs: 11.7 kB 1

@miguelmaso
miguelmaso merged commit bcb61c8 into main Oct 20, 2026
6 checks passed
@miguelmaso
miguelmaso deleted the version-parse branch October 20, 2026 07:28
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