Skip to content

Commit e2f4d18

Browse files
committed
disable type-stability tests for models on older Julia versions
1 parent bbdc060 commit e2f4d18

File tree

1 file changed

+27
-23
lines changed

1 file changed

+27
-23
lines changed

test/model.jl

Lines changed: 27 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -348,30 +348,34 @@ is_typed_varinfo(varinfo::DynamicPPL.SimpleVarInfo{<:NamedTuple}) = true
348348
end
349349
end
350350

351-
@testset "Type stability of models" begin
352-
models_to_test = [
353-
# FIXME: Fix issues with type-stability in `DEMO_MODELS`.
354-
# DynamicPPL.TestUtils.DEMO_MODELS...,
355-
DynamicPPL.TestUtils.demo_lkjchol(2),
356-
]
357-
@testset "$(model.f)" for model in models_to_test
358-
vns = DynamicPPL.TestUtils.varnames(model)
359-
example_values = DynamicPPL.TestUtils.rand(model)
360-
varinfos = filter(
361-
is_typed_varinfo,
362-
DynamicPPL.TestUtils.setup_varinfos(model, example_values, vns),
363-
)
364-
@testset "$(short_varinfo_name(varinfo))" for varinfo in varinfos
365-
@test (@inferred(DynamicPPL.evaluate!!(model, varinfo, DefaultContext()));
366-
true)
367-
368-
varinfo_linked = DynamicPPL.link(varinfo, model)
369-
@test (
370-
@inferred(
371-
DynamicPPL.evaluate!!(model, varinfo_linked, DefaultContext())
372-
);
373-
true
351+
if VERSION >= v"1.8"
352+
@testset "Type stability of models" begin
353+
models_to_test = [
354+
# FIXME: Fix issues with type-stability in `DEMO_MODELS`.
355+
# DynamicPPL.TestUtils.DEMO_MODELS...,
356+
DynamicPPL.TestUtils.demo_lkjchol(2),
357+
]
358+
@testset "$(model.f)" for model in models_to_test
359+
vns = DynamicPPL.TestUtils.varnames(model)
360+
example_values = DynamicPPL.TestUtils.rand(model)
361+
varinfos = filter(
362+
is_typed_varinfo,
363+
DynamicPPL.TestUtils.setup_varinfos(model, example_values, vns),
374364
)
365+
@testset "$(short_varinfo_name(varinfo))" for varinfo in varinfos
366+
@test (
367+
@inferred(DynamicPPL.evaluate!!(model, varinfo, DefaultContext()));
368+
true
369+
)
370+
371+
varinfo_linked = DynamicPPL.link(varinfo, model)
372+
@test (
373+
@inferred(
374+
DynamicPPL.evaluate!!(model, varinfo_linked, DefaultContext())
375+
);
376+
true
377+
)
378+
end
375379
end
376380
end
377381
end

0 commit comments

Comments
 (0)