Merge pull request #9 from wearepal/testsss

collect tests and make pass

Author: olliethomas

pyproject.toml

@@ -39,6 +39,9 @@ dev-dependencies = [
     "python-type-stubs @ git+https://github.com/wearepal/python-type-stubs@95a26e5",
 ]
 
+[tool.rye.scripts]
+rrm = {cmd="python run/rrm_credit_lr.py"}
+
 [tool.hatch.metadata]
 allow-direct-references = true
 
@@ -136,3 +139,15 @@ reportUnknownArgumentType = "none"
 reportUnknownLambdaType = "none"
 venvPath = "."
 venv = ".venv"
+
+[tool.pytest.ini_options]
+python_files = "*.py"
+python_classes = "Test"
+python_functions = "test_*"
+addopts = ["--import-mode=importlib"]
+# Ignore deprecation warnings coming from pytest/pkg_resources
+# that pollute the console output when testing.
+filterwarnings = [
+   "ignore::DeprecationWarning:_pytest.*:",
+   "ignore::DeprecationWarning:pkg_resources.*:"
+]

src/dynamics/response.py

@@ -32,7 +32,7 @@ def respond(
                 "'action' should correspond to the feature weights and thus must have entries "
                 "numbering the number of columns in 'features'"
             )
-        changeable_features = unwrap_or(self.changeable_features, default=slice(None))
+        changeable_features = unwrap_or(self.changeable_features, default=[])
         new_features = np.copy(features)
 
         neg_item_mask = np.dot(action[None, :], features[...].T)[0] < 0
@@ -164,7 +164,8 @@ def test_rir_response():
 
         response_fn = RIRResponse(epsilon=1.0, changeable_features=None)
         new_features = response_fn(features=features, action=action)
-        assert not np.testing.assert_array_equal(features, new_features)
+        with pytest.raises(AssertionError):
+            assert np.testing.assert_array_equal(features, new_features)
 
         changeable_features = np.array([0, 2])
         response_fn = RIRResponse(epsilon=1.0, changeable_features=changeable_features)