minor improvement

Author: findmyway

src/base.jl

@@ -190,3 +190,23 @@ function test_interfaces!(env)
 
     reset!(env)
 end
+
+function test_runnable!(env, n = 1000;rng=Random.GLOBAL_RNG)
+    @testset "random policy with $(nameof(env))" begin
+        reset!(env)
+        for _ in 1:n
+            A = legal_action_space(env)
+            a = rand(rng, A)
+            @test a in A
+
+            S = state_space(env)
+            s = state(env)
+            @test s in S
+            env(a)
+            if is_terminated(env)
+                reset!(env)
+            end
+        end
+        reset!(env)
+    end
+end

src/examples/TicTacToeEnv.jl

@@ -377,17 +377,17 @@ DefaultStateStyle(ss::Tuple{Vararg{<:AbstractStateStyle}}) = first(ss)
 # General
 #####
 
-struct DefaultPlayer end
-const DEFAULT_PLAYER = DefaultPlayer()
+@api struct DefaultPlayer end
+@api const DEFAULT_PLAYER = DefaultPlayer()
 
-struct ChancePlayer end
-const CHANCE_PLAYER = ChancePlayer()
+@api struct ChancePlayer end
+@api const CHANCE_PLAYER = ChancePlayer()
 
-struct SimultaneousPlayer end
-const SIMULTANEOUS_PLAYER = SimultaneousPlayer()
+@api struct SimultaneousPlayer end
+@api const SIMULTANEOUS_PLAYER = SimultaneousPlayer()
 
-struct Spector end
-const SPECTOR = Spector()
+@api struct Spector end
+@api const SPECTOR = Spector()
 
 @api (env::AbstractEnv)(action, player = current_player(env))
 
@@ -400,7 +400,7 @@ Make an independent copy of `env`,
 @api copy(env::AbstractEnv) = deepcopy(env)
 @api copyto!(dest::AbstractEnv, src::AbstractEnv)
 
-@env_api nameof(env::AbstractEnv) = nameof(typeof(env))
+@api nameof(env::AbstractEnv) = nameof(typeof(env))
 
 """
 Get the action distribution of chance player.