Skip to content

Commit

Permalink
Merge pull request #3542 from CliMA/dy/update_cts
Browse files Browse the repository at this point in the history
Update ClimaTimeSteppers to 0.8.1
  • Loading branch information
Sbozzolo authored Jan 30, 2025
2 parents 6cdeb57 + 88168a3 commit 2612ba7
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 12 deletions.
4 changes: 2 additions & 2 deletions .buildkite/Manifest-v1.11.toml
Original file line number Diff line number Diff line change
Expand Up @@ -418,9 +418,9 @@ version = "0.1.1"

[[deps.ClimaTimeSteppers]]
deps = ["ClimaComms", "Colors", "DataStructures", "DiffEqBase", "KernelAbstractions", "Krylov", "LinearAlgebra", "LinearOperators", "NVTX", "SciMLBase", "StaticArrays"]
git-tree-sha1 = "bd5ac9dfd8b38b86b37cebb0cd162c761906c3c0"
git-tree-sha1 = "f03e9f4316d380cdf851ec2c4c55efbfdb064439"
uuid = "595c0a79-7f3d-439a-bc5a-b232dc3bde79"
version = "0.7.40"
version = "0.8.1"
weakdeps = ["BenchmarkTools", "CUDA", "OrderedCollections", "PrettyTables", "StatsBase"]

[deps.ClimaTimeSteppers.extensions]
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/downgrade.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
version: ['1.9', '1.10', '1.11']
version: ['1.10', '1.11']
steps:
- uses: actions/checkout@v4
- uses: julia-actions/setup-julia@latest
Expand Down
4 changes: 2 additions & 2 deletions Project.toml
Original file line number Diff line number Diff line change
Expand Up @@ -42,10 +42,10 @@ ArtifactWrappers = "0.2"
Artifacts = "1"
AtmosphericProfilesLibrary = "0.1.7"
ClimaComms = "0.6.4"
ClimaCore = "0.14.19"
ClimaCore = "0.14.24"
ClimaDiagnostics = "0.2.4"
ClimaParams = "0.10.17"
ClimaTimeSteppers = "0.7.38"
ClimaTimeSteppers = "0.8.1"
ClimaUtilities = "0.1.14"
CloudMicrophysics = "0.22.3"
Dates = "1"
Expand Down
8 changes: 4 additions & 4 deletions perf/benchmark.jl
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,8 @@ device = ClimaComms.device(config.comms_ctx)
"T_exp_T_lim!",
# "lim!",
"dss!",
"post_explicit!",
"post_implicit!",
"cache!",
"cache_imp!",
"step!",
],
)
Expand All @@ -59,8 +59,8 @@ are_boundschecks_forced = Base.JLOptions().check_bounds == 1
@test compare_mem(trials, "T_exp_T_lim!", 9920)
@test compare_mem(trials, "lim!", 0)
@test compare_mem(trials, "dss!", 0)
@test compare_mem(trials, "post_explicit!", 120)
@test compare_mem(trials, "post_implicit!", 160)
@test compare_mem(trials, "cache!", 120)
@test compare_mem(trials, "cache_imp!", 160)

# It's difficult to guarantee zero allocations,
# so let's just leave this as broken for now.
Expand Down
6 changes: 3 additions & 3 deletions src/solver/type_getters.jl
Original file line number Diff line number Diff line change
Expand Up @@ -576,9 +576,9 @@ function args_integrator(parsed_args, Y, p, tspan, ode_algo, callback)
# Can we just pass implicit_tendency! and jac_prototype etc.?
lim! = limiters_func!,
dss!,
post_explicit! = set_precomputed_quantities!,
post_implicit! = set_precomputed_quantities!,
)
cache! = set_precomputed_quantities!,
cache_imp! = set_precomputed_quantities!,
) # TODO: Split implicit precomputed quantities from the rest.
else
SciMLBase.SplitFunction(implicit_func, remaining_tendency!)
end
Expand Down

0 comments on commit 2612ba7

Please sign in to comment.