Better

Author: gdalle

src/dense.jl

@@ -111,11 +111,11 @@ mode(::AutoFiniteDiff) = ForwardMode()
 function Base.show(io::IO, backend::AutoFiniteDiff)
     print(io, AutoFiniteDiff, "(")
     backend.fdtype != Val(:forward) &&
-        print(io, "fdtype=", repr(backend.fdtype; context = io), ",")
+        print(io, "fdtype=", repr(backend.fdtype; context = io), ", ")
     backend.fdjtype != backend.fdtype &&
-        print(io, "fdjtype=", repr(backend.fdjtype; context = io), ",")
+        print(io, "fdjtype=", repr(backend.fdjtype; context = io), ", ")
     backend.fdhtype != Val(:hcentral) &&
-        print(io, "fdhtype=", repr(backend.fdhtype; context = io), ",")
+        print(io, "fdhtype=", repr(backend.fdhtype; context = io))
     print(io, ")")
 end
 
@@ -175,8 +175,9 @@ mode(::AutoForwardDiff) = ForwardMode()
 
 function Base.show(io::IO, backend::AutoForwardDiff{chunksize}) where {chunksize}
     print(io, AutoForwardDiff, "(")
-    chunksize !== nothing && print(io, "chunksize=", repr(chunksize; context = io), ",")
-    backend.tag !== nothing && print(io, "tag=", repr(backend.tag; context = io), ",")
+    chunksize !== nothing && print(io, "chunksize=", repr(chunksize; context = io),
+        (backend.tag !== nothing ? ", " : ""))
+    backend.tag !== nothing && print(io, "tag=", repr(backend.tag; context = io))
     print(io, ")")
 end
 
@@ -211,8 +212,9 @@ mode(::AutoPolyesterForwardDiff) = ForwardMode()
 
 function Base.show(io::IO, backend::AutoPolyesterForwardDiff{chunksize}) where {chunksize}
     print(io, AutoPolyesterForwardDiff, "(")
-    chunksize !== nothing && print(io, "chunksize=", repr(chunksize; context = io), ",")
-    backend.tag !== nothing && print(io, "tag=", repr(backend.tag; context = io), ",")
+    chunksize !== nothing && print(io, "chunksize=", repr(chunksize; context = io),
+        (backend.tag !== nothing ? ", " : ""))
+    backend.tag !== nothing && print(io, "tag=", repr(backend.tag; context = io))
     print(io, ")")
 end
 

src/sparse.jl

@@ -155,13 +155,13 @@ function AutoSparse(
 end
 
 function Base.show(io::IO, backend::AutoSparse)
-    print(io, AutoSparse, "(dense_ad=", repr(backend.dense_ad, context = io), ",")
+    print(io, AutoSparse, "(dense_ad=", repr(backend.dense_ad, context = io))
     if backend.sparsity_detector != NoSparsityDetector()
-        print(io, "sparsity_detector=", repr(backend.sparsity_detector, context = io), ",")
+        print(io, ", sparsity_detector=", repr(backend.sparsity_detector, context = io))
     end
     if backend.coloring_algorithm != NoColoringAlgorithm()
         print(
-            io, "coloring_algorithm=", repr(backend.coloring_algorithm, context = io), ",")
+            io, ", coloring_algorithm=", repr(backend.coloring_algorithm, context = io))
     end
     print(io, ")")
 end

test/misc.jl

@@ -20,3 +20,40 @@ end
     @test contains(string(sparse_backend1), string(AutoForwardDiff()))
     @test length(string(sparse_backend1)) < length(string(sparse_backend2))
 end
+
+import ADTypes
+
+struct FakeSparsityDetector <: ADTypes.AbstractSparsityDetector end
+struct FakeColoringAlgorithm <: ADTypes.AbstractColoringAlgorithm end
+
+for backend in [
+    # dense
+    ADTypes.AutoChainRules(; ruleconfig = :rc),
+    ADTypes.AutoDiffractor(),
+    ADTypes.AutoEnzyme(),
+    ADTypes.AutoEnzyme(mode = :forward),
+    ADTypes.AutoFastDifferentiation(),
+    ADTypes.AutoFiniteDiff(),
+    ADTypes.AutoFiniteDiff(fdtype = :fd, fdjtype = :fdj, fdhtype = :fdh),
+    ADTypes.AutoFiniteDifferences(; fdm = :fdm),
+    ADTypes.AutoForwardDiff(),
+    ADTypes.AutoForwardDiff(chunksize = 3, tag = :tag),
+    ADTypes.AutoPolyesterForwardDiff(),
+    ADTypes.AutoPolyesterForwardDiff(chunksize = 3, tag = :tag),
+    ADTypes.AutoReverseDiff(),
+    ADTypes.AutoReverseDiff(compile = true),
+    ADTypes.AutoSymbolics(),
+    ADTypes.AutoTapir(),
+    ADTypes.AutoTapir(safe_mode = false),
+    ADTypes.AutoTracker(),
+    ADTypes.AutoZygote(),
+    # sparse
+    ADTypes.AutoSparse(ADTypes.AutoForwardDiff()),
+    ADTypes.AutoSparse(
+        ADTypes.AutoForwardDiff();
+        sparsity_detector = FakeSparsityDetector(),
+        coloring_algorithm = FakeColoringAlgorithm()
+    )
+]
+    println(backend)
+end