Don't use SCIP_Var in InitialSolution

include/scippp/initial_solution.hpp

@@ -2,9 +2,6 @@
 
 #include <map>
 
-// forward declare
-struct SCIP_Var;
-
 namespace scippp {
 
 // forward declare
@@ -18,7 +15,7 @@ class Var;
 class InitialSolution {
     friend class Model;
     //! Variable assignment in the initial solution.
-    std::map<SCIP_Var*, double> m_values {};
+    std::map<const Var*, double> m_values {};
 
 public:
     /**

source/initial_solution.cpp

@@ -1,18 +1,16 @@
 #include "scippp/initial_solution.hpp"
 
-#include <scippp/var.hpp>
-
 namespace scippp {
 
 void InitialSolution::setValue(const Var& var, double value)
 {
     // maybe add scip pointer to var and check this here
-    m_values[var.var] = value;
+    m_values[&var] = value;
 }
 
 double& InitialSolution::operator()(const Var& var)
 {
-    return m_values[var.var];
+    return m_values[&var];
 }
 
 }

source/model.cpp

@@ -161,7 +161,7 @@ bool Model::addSolution(
     SCIP_Sol* sol { nullptr };
     m_scipCallWrapper(SCIPcreateSol(m_scip, &sol, nullptr));
     for (const auto& [var, value] : initialSolution.m_values) {
-        m_scipCallWrapper(SCIPsetSolVal(m_scip, sol, var, value));
+        m_scipCallWrapper(SCIPsetSolVal(m_scip, sol, var->getVar(), value));
     }
     SCIP_Bool isFeasible { false };
     m_scipCallWrapper(SCIPcheckSol(