Skip to content

Melons lemons apples #20

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

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Melons lemons apples #20

wants to merge 18 commits into from

Conversation

beduffy
Copy link
Collaborator

@beduffy beduffy commented Mar 1, 2018

No description provided.

large positive reward, small negative reward and small positive reward respectively. Checks to see if params are correct.
Reset logic done correctly with lists
constants for the reward amounts
test environment txt file
test_lemons_melons_apples() tests cumulative reward is expected from collecting 9 different fruit and moving through environment
test_fruit_disappears() is self explanatory
test_fruit_reset() checks for fruit reappearance after reset
Also added self.MOVEMENT_REWARD set to -1 in case we want to change in the future
Created transparent background apple. Still some bugs. Added run_gridworld_with_fruit() example
images to be added in another commit once alpha background done properly
Created path to go up and down for interesting GIFs.
lava, terminal, movement, lemon, melon. apple, screen_width, screen_height.
Fixed one test case
@beduffy beduffy self-assigned this Mar 1, 2018
@beduffy beduffy requested a review from fernandotorch March 1, 2018 22:36
beduffy added 8 commits March 2, 2018 19:28
Was very tough and very error-prone. All tests broken initially. Fruits didn't show but now they do.
…agent code block inline so people can see fast how to use the environment
This env example is the first with parameters (one for random fill and one for grid_shape) which is a good idea
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant