From b948a45780e9a39f00643a80019d43ca6b062a81 Mon Sep 17 00:00:00 2001
From: Will Tebbutt <wt0881@my.bristol.ac.uk>
Date: Wed, 25 Sep 2024 11:53:25 +0100
Subject: [PATCH] Deprecate AutoTapir

---
 src/dense.jl  | 4 ++--
 src/legacy.jl | 7 ++++++-
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/src/dense.jl b/src/dense.jl
index 93ec5dd..fb4be49 100644
--- a/src/dense.jl
+++ b/src/dense.jl
@@ -351,8 +351,8 @@ mode(::AutoSymbolics) = SymbolicMode()
 
 Tapir.jl has been renamed to Mooncake.jl. Please use `AutoMooncake` instead of this type.
 """
-Base.@kwdef struct AutoTapir <: AbstractADType
-    safe_mode::Bool = true
+struct AutoTapir <: AbstractADType
+    safe_mode::Bool
 end
 
 mode(::AutoTapir) = ReverseMode()
diff --git a/src/legacy.jl b/src/legacy.jl
index 46a5214..d9dd942 100644
--- a/src/legacy.jl
+++ b/src/legacy.jl
@@ -37,4 +37,9 @@ function AutoModelingToolkit(; obj_sparse::Bool = false, cons_sparse::Bool = fal
     return mtk_to_symbolics(obj_sparse, cons_sparse)
 end
 
-@deprecate AutoTapir AutoMooncake
+function AutoTapir(; safe_mode=true)
+    Base.depwarn(
+        "AutoTapir is deprecated in favour of AutoMooncake.", :AutoTapir; force=false
+    )
+    return AutoTapir(safe_mode)
+end