Expose the knitro context (#600)

* code: expose knitro context and modify _extract_values

* doc: update release_notes.rst

* code: include pre-commit checks

Author: finozzifa

doc/release_notes.rst

@@ -4,6 +4,9 @@ Release Notes
 Upcoming Version
 ----------------
 
+* Expose the knitro context to allow for more flexible use of the knitro python API.
+
+
 Version 0.6.4
 --------------
 

linopy/solvers.py

@@ -1745,7 +1745,7 @@ def get_solver_solution() -> Solution:
         return Result(status, solution, m)
 
 
-KnitroResult = namedtuple("KnitroResult", "reported_runtime")
+KnitroResult = namedtuple("KnitroResult", "knitro_context reported_runtime")
 
 
 class Knitro(Solver[None]):
@@ -1808,7 +1808,13 @@ def _extract_values(
         if n == 0:
             return pd.Series(dtype=float)
 
-        values = get_values_fn(kc, n - 1)
+        try:
+            # Compatible with KNITRO >= 15
+            values = get_values_fn(kc)
+        except TypeError:
+            # Fallback for older wrappers requiring explicit indices
+            values = get_values_fn(kc, list(range(n)))
+
         names = list(get_names_fn(kc))
         return pd.Series(values, index=names, dtype=float)
 
@@ -1931,12 +1937,14 @@ def get_solver_solution() -> Solution:
                 knitro.KN_write_mps_file(kc, path_to_string(solution_fn))
 
             return Result(
-                status, solution, KnitroResult(reported_runtime=reported_runtime)
+                status,
+                solution,
+                KnitroResult(knitro_context=kc, reported_runtime=reported_runtime),
             )
 
         finally:
-            with contextlib.suppress(Exception):
-                knitro.KN_free(kc)
+            # Intentionally keep the Knitro context alive; do not free `kc` here.
+            pass
 
 
 mosek_bas_re = re.compile(r" (XL|XU)\s+([^ \t]+)\s+([^ \t]+)| (LL|UL|BS)\s+([^ \t]+)")