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.

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

Replace LegacyGridWorld with SimpleGridWorld #89

Merged
merged 3 commits into from
Feb 12, 2022

Conversation

BoZenKhaa
Copy link
Contributor

I replaced LegacyGridWorld with SimpleGridWorld in tests and benchmarks. For the most part, I simple replaced the names of the corresponding structures.

This seems to mostly work, with few exceptions:

  • in the TestVisualization.ipynb, its no longer possible to mark terminal states in the tree, since MCTS.node_tag does not take the mdp parameter needed for isterminal(mdp, state). Previously, this worked since LegacyGridWorld state had field done, accessible from anywhere.

  • There is a test in runtest.jl that I believe used to tests whether a transition from terminal state throws an error. That doesn't happen with SimpleGridWorld. I have commented the test out for now.

  • I can't get ProfileView.jl to run on my headles server where I use Julia, so I have commented it's use out in the bench scripts.

Also, I have removed the REQUIRE file and replaced it with Project.toml for the test directory and updated the .gitignore correspondingly.

This should address these issues: #57, #59, and possibly #62

Copy link
Member

@zsunberg zsunberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, thank you for all of this! I am on vacation from now until Monday, but I will try to merge it as soon as I get back.

@zsunberg
Copy link
Member

@BoZenKhaa thanks! Looks good! It is funny to see how old those scripts are.

@zsunberg zsunberg merged commit f6ed9e2 into JuliaPOMDP:master Feb 12, 2022
@BoZenKhaa
Copy link
Contributor Author

Great, thanks! I think it's ok to close the related issues too.

@BoZenKhaa BoZenKhaa deleted the new_gw branch April 16, 2024 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants