now raise error when covariance type other than diag is passed to GMM (instead of warning and mutating like currently done)

Author: MechaCritter

pyvisim/clustering/gmm.py

@@ -1,6 +1,5 @@
 """Gaussian Mixture Model used by the Fisher Vector encoder."""
 
-import warnings
 from typing import Any
 
 import numpy as np
@@ -40,12 +39,10 @@ def __init__(self, n_components: int = 256, **gmm_params: Any) -> None:
 
     def _validate_sklearn_model(self) -> None:
         if self._model.covariance_type != "diag":
-            warnings.warn(
-                "Attribute 'covariance_type' of the underlying GaussianMixture is "
-                "set to 'diag' because training will take too long otherwise.",
-                stacklevel=2,
+            raise ValueError(
+                f"{type(self).__name__} only supports covariance_type='diag', "
+                f"got {self._model.covariance_type!r}."
             )
-            self._model.covariance_type = "diag"
 
     @property
     def n_clusters(self) -> int: