Merge pull request #755 from JuliaSymbolics/myb/getname

Handle getname edge case
JuliaSymbolics · Oct 5, 2022 · 1b50175 · 1b50175
2 parents 54b583c + db1e1d0
commit 1b50175
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/variable.jl b/src/variable.jl
@@ -390,6 +390,7 @@ const _fail = Dict()
 _getname(x, _) = nameof(x)
 _getname(x::Symbol, _) = x
 function _getname(x::Symbolic, val)
+    issym(x) && return nameof(x)
     if istree(x) && issym(operation(x))
         return nameof(operation(x))
     end

diff --git a/test/arrays.jl b/test/arrays.jl
@@ -1,6 +1,6 @@
 using Symbolics
 using SymbolicUtils, Test
-using Symbolics: symtype, shape, wrap, unwrap, Unknown, Arr, arrterm, jacobian, @variables, value, get_variables, @arrayop
+using Symbolics: symtype, shape, wrap, unwrap, Unknown, Arr, arrterm, jacobian, @variables, value, get_variables, @arrayop, getname
 using Base: Slice
 using SymbolicUtils: Sym, term, operation
 
@@ -29,6 +29,12 @@ using SymbolicUtils: Sym, term, operation
     @test isequal(get_variables(2x[1]), [x[1]])
 end
 
+@testset "getname" begin
+    @variables t x(t)[1:4]
+    v = Symbolics.lower_varname(unwrap(x[2]), unwrap(t), 2)
+    @test getname(v) == Symbol("x(t)[2]ˍtt")
+end
+
 @testset "getindex" begin
     @variables X[1:5, 1:5] Y[1:5, 1:5]