fix: various bug and test fixes

Author: AayushSabharwal

src/systems/abstractsystem.jl

@@ -482,10 +482,12 @@ function SymbolicIndexingInterface.parameter_index(sys::AbstractSystem, sym)
             end
         elseif iscall(sym) && operation(sym) === getindex &&
                (idx = parameter_index(ic, first(arguments(sym)))) !== nothing
-            if idx.portion isa SciMLStructures.Discrete && idx.idx[2] == idx.idx[3] == nothing
+            if idx.portion isa SciMLStructures.Discrete &&
+               idx.idx[2] == idx.idx[3] == nothing
                 return nothing
             else
-                ParameterIndex(idx.portion, (idx.idx..., arguments(sym)[(begin + 1):end]...))
+                ParameterIndex(
+                    idx.portion, (idx.idx..., arguments(sym)[(begin + 1):end]...))
             end
         else
             nothing
@@ -505,7 +507,8 @@ end
 function SymbolicIndexingInterface.parameter_index(sys::AbstractSystem, sym::Symbol)
     if has_index_cache(sys) && (ic = get_index_cache(sys)) !== nothing
         idx = parameter_index(ic, sym)
-        if idx === nothing || idx.portion isa SciMLStructures.Discrete && idx.idx[2] == idx.idx[3] == 0
+        if idx === nothing ||
+           idx.portion isa SciMLStructures.Discrete && idx.idx[2] == idx.idx[3] == 0
             return nothing
         else
             return idx

src/systems/parameter_buffer.jl

@@ -158,7 +158,7 @@ function MTKParameters(
         end
     end
     tunable_buffer = narrow_buffer_type.(tunable_buffer)
-    disc_buffer = narrow_buffer_type.(disc_buffer)
+    disc_buffer = broadcast.(narrow_buffer_type, disc_buffer)
     const_buffer = narrow_buffer_type.(const_buffer)
     # Don't narrow nonnumeric types
     nonnumeric_buffer = nonnumeric_buffer
@@ -568,7 +568,7 @@ function SymbolicIndexingInterface.remake_buffer(indp, oldbuf::MTKParameters, va
         @set! newbuf.dependent = narrow_buffer_type_and_fallback_undefs.(
             oldbuf.dependent,
             split_into_buffers(
-                newbuf.dependent_update_oop(newbuf...), oldbuf.dependent, Val(false)))
+                newbuf.dependent_update_oop(newbuf...), oldbuf.dependent, Val(0)))
     end
     return newbuf
 end

test/mtkparameters.jl

@@ -321,8 +321,8 @@ with_updated_parameter_timeseries_values(
 @test SciMLBase.get_saveable_values(ps, 1).x == ps.discrete[1]
 
 # With multiple types and clocks
-ps = MTKParameters((), SizedVector{2}([([1.0, 2.0, 3.0], [false]), ([4.0, 5.0, 6.0], Bool[])]), (), (), (), nothing, nothing)
-@test SciMLBase.get_saveable_values(ps, 1).x isa Tuple{Vector{Float64}, Vector{Bool}}
+ps = MTKParameters((), SizedVector{2}([([1.0, 2.0, 3.0], falses(1)), ([4.0, 5.0, 6.0], falses(0))]), (), (), (), nothing, nothing)
+@test SciMLBase.get_saveable_values(ps, 1).x isa Tuple{Vector{Float64}, BitVector}
 tsidx1 = 1
 tsidx2 = 2
 @test length(ps.discrete[tsidx1][1]) == 3

test/symbolic_indexing_interface.jl

@@ -19,7 +19,8 @@ using SciMLStructures: Tunable
     @test parameter_index(odesys, a) isa ParameterIndex{Tunable, Tuple{Int, Int}}
     @test parameter_index(odesys, b) == parameter_index(odesys, :b)
     @test parameter_index(odesys, b) isa ParameterIndex{Tunable, Tuple{Int, Int}}
-    @test parameter_index.((odesys,), [x, y, t, ParameterIndex(Tunable(), (1, 1)), :x, :y,]) ==
+    @test parameter_index.(
+        (odesys,), [x, y, t, ParameterIndex(Tunable(), (1, 1)), :x, :y]) ==
           [nothing, nothing, nothing, ParameterIndex(Tunable(), (1, 1)), nothing, nothing]
     @test isequal(parameter_symbols(odesys), [a, b])
     @test all(is_independent_variable.((odesys,), [t, :t]))