fix tests with step returning np.bool_

gym/envs/mujoco/inverted_pendulum.py

@@ -15,7 +15,7 @@ def step(self, a):
         reward = 1.0
         self.do_simulation(a, self.frame_skip)
         ob = self._get_obs()
-        terminated = not np.isfinite(ob).all() or (np.abs(ob[1]) > 0.2)
+        terminated = bool(not np.isfinite(ob).all() or (np.abs(ob[1]) > 0.2))
         return ob, reward, terminated, False, {}
 
     def reset_model(self):

gym/envs/mujoco/inverted_pendulum_v4.py

@@ -98,7 +98,7 @@ def step(self, a):
         reward = 1.0
         self.do_simulation(a, self.frame_skip)
         ob = self._get_obs()
-        terminated = not np.isfinite(ob).all() or (np.abs(ob[1]) > 0.2)
+        terminated = bool(not np.isfinite(ob).all() or (np.abs(ob[1]) > 0.2))
         return ob, reward, terminated, False, {}
 
     def reset_model(self):

gym/envs/registration.py

@@ -23,7 +23,12 @@
 import numpy as np
 
 from gym.envs.__relocated__ import internal_env_relocation_map
-from gym.wrappers import AutoResetWrapper, OrderEnforcing, StepAPICompatibility, TimeLimit
+from gym.wrappers import (
+    AutoResetWrapper,
+    OrderEnforcing,
+    StepAPICompatibility,
+    TimeLimit,
+)
 from gym.wrappers.env_checker import PassiveEnvChecker
 
 if sys.version_info < (3, 10):
@@ -602,7 +607,7 @@ def make(
     # Run the environment checker as the lowest level wrapper
     if disable_env_checker is False:
         env = PassiveEnvChecker(env)
-        
+
     env = StepAPICompatibility(env, new_step_api)
 
     # Add the order enforcing wrapper
@@ -619,15 +624,6 @@ def make(
     if autoreset:
         env = AutoResetWrapper(env, new_step_api)
 
-    if not disable_env_checker:
-        try:
-            check_env(env)
-        except Exception as e:
-            logger.warn(
-                f"Env check failed with the following message: {e}\n"
-                f"You can set `disable_env_checker=True` to disable this check."
-            )
-
     return env
 
 

gym/utils/env_checker.py

@@ -27,7 +27,6 @@
     passive_env_reset_check,
     passive_env_step_check,
 )
-from gym.utils.step_api_compatibility import step_api_compatibility
 
 
 def data_equivalence(data_1, data_2) -> bool:

gym/utils/passive_env_checker.py

@@ -255,12 +255,18 @@ def passive_env_step_check(env, action):
     if len(result) == 4:
         obs, reward, done, info = result
 
-        assert isinstance(done, bool), "The `done` signal must be a boolean"
+        assert isinstance(
+            done, bool
+        ), f"The `done` signal is of type `{type(done)}` must be a boolean"
     elif len(result) == 5:
         obs, reward, terminated, truncated, info = result
 
-        assert isinstance(terminated, bool), "The `terminated` signal must be a boolean"
-        assert isinstance(truncated, bool), "The `truncated` signal must be a boolean"
+        assert isinstance(
+            terminated, bool
+        ), f"The `terminated` signal is of type `{type(terminated)}`. It must be a boolean"
+        assert isinstance(
+            truncated, bool
+        ), f"The `truncated` signal of type `{type(truncated)}`. It must be a boolean."
         assert (
             terminated is False or truncated is False
         ), "Only `terminated` or `truncated` can be true, not both."

gym/vector/__init__.py

@@ -48,7 +48,10 @@ def create_env(_disable_env_checker):
 
         def _make_env():
             env = gym.envs.registration.make(
-                id, disable_env_checker=_disable_env_checker, new_step_api=True, **kwargs
+                id,
+                disable_env_checker=_disable_env_checker,
+                new_step_api=True,
+                **kwargs,
             )
             if wrappers is not None:
                 if callable(wrappers):
@@ -68,4 +71,8 @@ def _make_env():
         create_env(env_num == 0 and disable_env_checker is False)
         for env_num in range(num_envs)
     ]
-    return AsyncVectorEnv(env_fns, new_step_api=new_step_api) if asynchronous else SyncVectorEnv(env_fns, new_step_api=new_step_api)
+    return (
+        AsyncVectorEnv(env_fns, new_step_api=new_step_api)
+        if asynchronous
+        else SyncVectorEnv(env_fns, new_step_api=new_step_api)
+    )