Fix minor bugs: nsteps is not an argument to the ModelList anymore, and verbose to the ModelList method of dep

Author: VEZY

src/component_models/ModelList.jl

@@ -328,15 +328,17 @@ function Base.copy(m::T) where {T<:ModelList}
     ModelList(
         m.models,
         deepcopy(m.status),
-        deepcopy(m.type_promotion)
+        deepcopy(m.type_promotion),
+        deepcopy(m.dependency_graph)
     )
 end
 
 function Base.copy(m::T, status) where {T<:ModelList}
     ModelList(
         m.models,
         status,
-        deepcopy(m.type_promotion)
+        deepcopy(m.type_promotion),
+        deepcopy(m.dependency_graph)
     )
 end
 
@@ -468,7 +470,7 @@ function convert_vars!(mapped_vars::Dict{String,Dict{Symbol,Any}}, type_promotio
 end
 
 function Base.show(io::IO, ::MIME"text/plain", t::ModelList)
-    print(io, dep(t, verbose=false))
+    print(io, dep(t))
     print(io, status(t))
 end
 

src/processes/model_initialisation.jl

@@ -245,7 +245,7 @@ function init_variables(model::T; verbose::Bool=true) where {T<:AbstractModel}
 end
 
 function init_variables(m::ModelList; verbose::Bool=true)
-    init_variables(dep(m; verbose=verbose))
+    init_variables(dep(m))
 end
 
 function init_variables(m::DependencyGraph)

test/test-ModelList.jl

@@ -18,7 +18,6 @@
     leaf = ModelList(
         process1=Process1Model(1.0),
         process2=Process2Model(),
-        nsteps=3
     )
 
     @test length(status(leaf)) == 5
@@ -81,7 +80,6 @@ end;
         process1=Process1Model(1.0),
         process2=Process2Model(),
         status=(var1=15.0,),
-        nsteps=3
     )
 
     @test length(status(leaf)) == 5
@@ -123,8 +121,7 @@ end;
     inits = init_variables(leaf)
     sorted_vars = sort([keys(inits.process3)...])
 
-    @test [getfield(inits.process3, i) for i in sorted_vars] ==
-          fill(-Inf, 3)
+    @test [getfield(inits.process3, i) for i in sorted_vars] == fill(-Inf, 3)
 end;
 
 @testset "Copy a ModelList" begin