Merge pull request #2816 from SciML/b/change-arguments

Refactor: Migrate from deprecated `unsorted_arguments` to `arguments`

Author: ChrisRackauckas

Project.toml

@@ -108,7 +108,7 @@ SparseArrays = "1"
 SpecialFunctions = "0.7, 0.8, 0.9, 0.10, 1.0, 2"
 StaticArrays = "0.10, 0.11, 0.12, 1.0"
 SymbolicIndexingInterface = "0.3.12"
-SymbolicUtils = "2"
+SymbolicUtils = "2.1"
 Symbolics = "5.30.1"
 URIs = "1"
 UnPack = "0.1, 1.0"

src/systems/connectors.jl

@@ -85,7 +85,7 @@ function collect_instream!(set, expr, occurs = false)
     iscall(expr) || return occurs
     op = operation(expr)
     op === instream && (push!(set, expr); occurs = true)
-    for a in SymbolicUtils.unsorted_arguments(expr)
+    for a in SymbolicUtils.arguments(expr)
         occurs |= collect_instream!(set, a, occurs)
     end
     return occurs

src/utils.jl

@@ -281,7 +281,7 @@ function _check_operator_variables(eq, op::T, expr = eq.rhs) where {T}
         throw_invalid_operator(expr, eq, op)
     end
     foreach(expr -> _check_operator_variables(eq, op, expr),
-        SymbolicUtils.unsorted_arguments(expr))
+        SymbolicUtils.arguments(expr))
 end
 """
 Check if all the LHS are unique