Skip to content

Commit

Permalink
fixup! fix: convert Symbol to symbolic variables in remake, fix spl…
Browse files Browse the repository at this point in the history
…it_idxs
  • Loading branch information
AayushSabharwal committed Jan 3, 2024
1 parent 88530ce commit 4ec01be
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/variables.jl
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ function SciMLBase.process_p_u0_symbolic(prob::Union{SciMLBase.AbstractDEProblem
throw(ArgumentError("This problem does not support symbolic maps with `remake`, i.e. it does not have a symbolic origin." *
" Please use `remake` with the `p` keyword argument as a vector of values, paying attention to parameter order."))
p = [

Check warning on line 145 in src/variables.jl

View check run for this annotation

Codecov / codecov/patch

src/variables.jl#L145

Added line #L145 was not covered by tests
(sym isa Symbol ? parameter_symbols(prob)[parameter_index(prob, sym)] : sym) => val
(symbolic_type(sym) != NotSymbolic() ? parameter_symbols(prob)[parameter_index(prob, sym)] : sym) => val
for (sym, val) in p
]
end
Expand All @@ -152,7 +152,7 @@ function SciMLBase.process_p_u0_symbolic(prob::Union{SciMLBase.AbstractDEProblem
throw(ArgumentError("This problem does not support symbolic maps with `remake`, i.e. it does not have a symbolic origin." *
" Please use `remake` with the `u0` keyword argument as a vector of values, paying attention to state order."))
u0 = [

Check warning on line 154 in src/variables.jl

View check run for this annotation

Codecov / codecov/patch

src/variables.jl#L154

Added line #L154 was not covered by tests
(sym isa Symbol ? variable_symbols(prob)[variable_index(prob, sym)] : sym) => val
(symbolic_type(sym) != NotSymbolic() ? variable_symbols(prob)[variable_index(prob, sym)] : sym) => val
for (sym, val) in u0
]
end
Expand Down

0 comments on commit 4ec01be

Please sign in to comment.