Skip to content

Commit 34ab871

Browse files
committed
compelteness error for SI
1 parent f28ec30 commit 34ab871

File tree

3 files changed

+2
-54
lines changed

3 files changed

+2
-54
lines changed

ext/CatalystStructuralIdentifiabilityExtension/structural_identifiability_extension.jl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,7 @@ end
157157
function make_osys(rs::ReactionSystem; remove_conserved=true)
158158
# Creates the ODESystem corresponding to the ReactionSystem (expanding functions and flattening it).
159159
# Creates a list of the systems all symbols (unknowns and parameters).
160-
iscomplete(rs) || error(COMPLETENESS_ERROR)
160+
ModelingToolkit.iscomplete(rs) || error("Identifiability should only be computed for complete systems. A ReactionSystem can be marked as complete using the `complete` function.")
161161
rs = Catalyst.expand_registered_functions(flatten(rs))
162162
osys = complete(convert(ODESystem, rs; remove_conserved))
163163
vars = [unknowns(rs); parameters(rs)]

test/model_simulation/simulate_SDEs.jl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -282,7 +282,7 @@ let
282282
end
283283

284284
# Tests using complicated noise scaling expressions.
285-
@time let
285+
let
286286
noise_scaling_network = @reaction_network begin
287287
@parameters η1 η2 η3 η4
288288
@species N1(t) N2(t)=0.5

test/runtests.jl

Lines changed: 0 additions & 52 deletions
Original file line numberDiff line numberDiff line change
@@ -3,58 +3,6 @@ using SafeTestsets
33

44
### Run the tests ###
55
@time begin
6-
7-
### Tests the properties of ReactionSystems. ###
8-
@time @safetestset "Reactions" begin include("reactionsystem_structure/reactions.jl") end
9-
@time @safetestset "ReactionSystem" begin include("reactionsystem_structure/reactionsystem.jl") end
10-
@time @safetestset "Higher Order Reactions" begin include("reactionsystem_structure/higher_order_reactions.jl") end
11-
12-
### Tests model creation via the @reaction_network DSL. ###
13-
@time @safetestset "Basic DSL" begin include("dsl/dsl_basics.jl") end
14-
@time @safetestset "DSL Model Construction" begin include("dsl/dsl_model_construction.jl") end
15-
@time @safetestset "Custom CRN Functions" begin include("dsl/custom_functions.jl") end
16-
@time @safetestset "DSL Options" begin include("dsl/dsl_options.jl") end
17-
18-
### Non-DSL model creation and modification. ###
19-
@time @safetestset "ReactionSystem Components Based Creation" begin include("programmatic_model_creation/component_based_model_creation.jl") end
20-
@time @safetestset "Programmatic Model Expansion" begin include("programmatic_model_creation/programmatic_model_expansion.jl") end
21-
22-
# Runs various miscellaneous tests.
23-
@time @safetestset "API" begin include("miscellaneous_tests/api.jl") end
24-
@time @safetestset "Symbolic Stoichiometry" begin include("miscellaneous_tests/symbolic_stoichiometry.jl") end
25-
@time @safetestset "NonlinearProblems and Steady State Solving" begin include("miscellaneous_tests/nonlinear_solve.jl") end
26-
@time @safetestset "Events" begin include("miscellaneous_tests/events.jl") end
27-
@time @safetestset "Compound species" begin include("miscellaneous_tests/compound_macro.jl") end
28-
@time @safetestset "Reaction balancing" begin include("miscellaneous_tests/reaction_balancing.jl") end
29-
@time @safetestset "Units" begin include("miscellaneous_tests/units.jl") end
30-
31-
### Reaction network analysis. ###
32-
@time @safetestset "Conservation Laws" begin include("network_analysis/conservation_laws.jl") end
33-
@time @safetestset "Network Properties" begin include("network_analysis/network_properties.jl") end
34-
35-
### Tests ODE, SDE, PDE, and Gillespie Simulations. ###
36-
@time @safetestset "ODE System Simulations" begin include("model_simulation/simulate_ODEs.jl") end
37-
@time @safetestset "Automatic Jacobian Construction" begin include("model_simulation/make_jacobian.jl") end
38-
@time @safetestset "U0 and Parameters Input Variants" begin include("model_simulation/u0_n_parameter_inputs.jl") end
39-
@time @safetestset "SDE System Simulations" begin include("model_simulation/simulate_SDEs.jl") end
40-
@time @safetestset "Jump System Simulations" begin include("model_simulation/simulate_jumps.jl") end
41-
42-
### Tests Spatial Network Simulations. ###
43-
@time @safetestset "PDE Systems Simulations" begin include("spatial_reaction_systems/simulate_PDEs.jl") end
44-
@time @safetestset "Lattice Reaction Systems" begin include("spatial_reaction_systems/lattice_reaction_systems.jl") end
45-
@time @safetestset "ODE Lattice Systems Simulations" begin include("spatial_reaction_systems/lattice_reaction_systems_ODEs.jl") end
46-
47-
### Tests network visualization. ###
48-
@time @safetestset "Latexify" begin include("visualization/latexify.jl") end
49-
# @time @safetestset "Basic Plotting" begin include("visualization/plotting.jl") end
50-
# Disable on Macs as can't install GraphViz via jll
51-
if !Sys.isapple()
52-
@time @safetestset "Graphs" begin include("visualization/graphs.jl") end
53-
end
54-
55-
### Tests extensions. ###
56-
@time @safetestset "BifurcationKit Extension" begin include("extensions/bifurcation_kit.jl") end
57-
@time @safetestset "HomotopyContinuation Extension" begin include("extensions/homotopy_continuation.jl") end
586
@time @safetestset "Structural Identifiability Extension" begin include("extensions/structural_identifiability.jl") end
597

608
end # @time

0 commit comments

Comments
 (0)