JuliaML/Reinforce.jl

undefined function in CartPole environment

Closed this issue · 4 comments

Not sure that the function state(env) is defined in line 50 & 70 of the CartPoleEnv module.
It should probably replaced by env.state ! Am I wrong ?

Reinforce.jl/src/envs/cartpole.jl

https://github.com/JuliaML/Reinforce.jl#environment-interface
The state function is the standard interface of a AbstractEvnironment,
so you do not need to replace them.

It raises
ERROR: LoadError: UndefVarError: state not defined

We should then consider adding state in the imports at the beginning of the module:
At this point, we have:
import Reinforce: reset!, actions, finished, step! and we should move to:
import Reinforce: reset!, actions, finished, step!, state

Ah, you're right.
Could you send a PR about this issue?