From 551237e11bd3b92e35af8e8ba0fbabf688d30a3d Mon Sep 17 00:00:00 2001
From: Timo Koch <timok@simula.no>
Date: Thu, 27 Feb 2025 17:10:51 +0100
Subject: [PATCH] [properties][cleanup] Remove property LinearSolver

---
 CHANGELOG.md               | 3 +++
 dumux/common/properties.hh | 3 ---
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 8f83b44378..1884a63c3a 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -17,6 +17,9 @@ C++20 mode is enforced through CMake.
 - __Multidomain boundary__: Added the coupling condition and a test case for free-flow pore network single-phase composional non/isothermal flow.
 
 ### Immediate interface changes not allowing/requiring a deprecation period:
+-__Property LinearSolver__: Property `LinearSolver` has been removed. It was not used in Dumux and
+only a left-over from old IMPES-style models. The linear solver type is chosen in the main file and
+idenpendently of the property system.
 
 - __Property DefaultModelParameters__: Has been removed without replacement. Introduced in 3.0, we are not aware of any use case so far. If you have used this feature, please get in touch with us. This feature can easily be realized by a simple traits mechanism.
 
diff --git a/dumux/common/properties.hh b/dumux/common/properties.hh
index cc4f13eb63..9f0ba3dcd0 100644
--- a/dumux/common/properties.hh
+++ b/dumux/common/properties.hh
@@ -45,9 +45,6 @@ DUMUX_DEFINE_PROPERTY(SolutionVector)         //!< Vector containing all primary
 //! from the BaseLocalResidual property and fulfill its interfaces.
 DUMUX_DEFINE_PROPERTY(LocalResidual)
 
-//! TODO: Remove this property as soon as the decoupled models are integrated
-DUMUX_DEFINE_PROPERTY(LinearSolver)
-
 ////////////////////////////////////////////////
 // Basic properties regarding balance equations
 /////////////////////////////////////////////////
-- 
GitLab