Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

loadenvironment error #19

Closed
tejank10 opened this issue Sep 7, 2018 · 3 comments
Closed

loadenvironment error #19

tejank10 opened this issue Sep 7, 2018 · 3 comments

Comments

@tejank10
Copy link
Member

tejank10 commented Sep 7, 2018

Hi! I was playing with the package, trying out the usage examples from the docs. I am getting UndefVarError for example 2 on line 3, which says loadenvironment("cartpole")

ERROR: UndefVarError: loadenvironment not defined
Stacktrace:
 [1] top-level scope at none:0

I also could not find any reference to loadenvironment function in the codebase, except for in docs. Is it replaced with new function? Any hints?
@findmyway
Copy link
Member

Hi @tejank10

Glad to see you here 😄

@jbrea made some modifications recently. The document seems also need to be updated(but I guess he is really busy in the next few days.). AFAIK @jbrea just updated the examples of each supported environment yesterday, you may take a look at the example folder of each environment project there.

Notice that @jbrea 's PR are not merged yet. I'll update those PR and merge them soon.

  1. https://github.com/JuliaReinforcementLearning/ReinforcementLearningEnvironmentClassicControl.jl/pull/2/files

@jbrea
Copy link
Collaborator

jbrea commented Sep 13, 2018

The docs were outdated, thanks @tejank10. They are updated now, but the examples don't work with master until PRs #17, #18 are resolved.

@findmyway
Copy link
Member

Closing this issue since loadenvironment is now deprecated.

findmyway added a commit to findmyway/ReinforcementLearning.jl that referenced this issue May 3, 2021
@github-actions @findmyway
CompatHelper: add new compat entry for "MacroTools" at version "0.5" (J…
91e701c 
…uliaReinforcementLearning#19)

Co-authored-by: Jun Tian <[email protected]>
findmyway added a commit that referenced this issue May 3, 2021
@findmyway
Api change (#19)
243ce2c 
* unify interfaces

* update screen after interact

* add minor comment

* add version check

* update docker

* revert to juali v1.2 due to https://github.com/JuliaLang/julia/pull/32408\#issuecomment-522168938

* update README
findmyway pushed a commit that referenced this issue May 3, 2021
@JuliaTagBot
Install TagBot as a GitHub Action (#19)
73a36b5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@findmyway @jbrea @tejank10