Add comments re has_keops

Author: Ashley Scillitoe

alibi_detect/saving/tests/models.py

@@ -26,7 +26,7 @@
 from alibi_detect.cd.pytorch import HiddenOutput as HiddenOutput_pt
 from alibi_detect.cd.tensorflow import HiddenOutput as HiddenOutput_tf
 from alibi_detect.utils.frameworks import has_keops
-if has_keops:
+if has_keops:  # pykeops only installed in Linux CI
     from alibi_detect.utils.keops.kernels import GaussianRBF as GaussianRBF_ke
     from alibi_detect.utils.keops.kernels import DeepKernel as DeepKernel_ke
 

alibi_detect/saving/tests/test_saving.py

@@ -46,7 +46,7 @@
 from alibi_detect.utils.pytorch.kernels import DeepKernel as DeepKernel_pt
 from alibi_detect.utils.tensorflow.kernels import DeepKernel as DeepKernel_tf
 from alibi_detect.utils.frameworks import has_keops
-if has_keops:
+if has_keops:  # pykeops only installed in Linux CI
     from pykeops.torch import LazyTensor
     from alibi_detect.utils.keops.kernels import DeepKernel as DeepKernel_ke
 
@@ -56,7 +56,7 @@
 # TODO: We currently parametrize encoder_model etc (in models.py) with backend, so the same flavour of
 # preprocessing is used as the detector backend. In the future we could decouple this in tests.
 backends = ['tensorflow', 'pytorch', 'sklearn']
-if has_keops:  # keops currently only installed during linux CI
+if has_keops:  # pykeops only installed in Linux CI
     backends.append('keops')
 backend = param_fixture("backend", backends)
 P_VAL = 0.05