Retro-Learning-Environment icon indicating copy to clipboard operation
Retro-Learning-Environment copied to clipboard

game_over() set to True when Mario gets a Mushroom in Super Mario World

Open folivetti opened this issue 8 years ago • 6 comments

Whenever Mario hits a mushroom (and possibly any other item) the game returns True to game_over().

folivetti avatar Feb 15 '18 20:02 folivetti

If you change these lines:

https://github.com/nadavbh12/Retro-Learning-Environment/blob/0269da8773ac9fe267da14eda1146c40af14a7d4/src/games/supported/SuperMarioWorld.cpp#L77-L84

To

if (readRam(&system, 0xdda) == 0xff) { m_terminal=true; }

it works

folivetti avatar Feb 17 '18 12:02 folivetti

Thanks. Looks like it does fix it.

Though it seems the starting actions are broken. I vaguely remember that the first set of actions doesn't work once you start a game because of the internal saves in Mario. Moving to save states should solve these issues.

nadavbh12 avatar Feb 28 '18 08:02 nadavbh12

Though it seems the starting actions are broken.

it seems fine to me. Do you mean the initial inputs are not processed or the actions to start the game are not performed?

folivetti avatar Feb 28 '18 12:02 folivetti

Once mario dies, I go to Yoshi's island rather than the beginning of the level. The starting_actions for the first time you run the game, aren't the same for all subsequent restarts.

nadavbh12 avatar Feb 28 '18 13:02 nadavbh12

that already happened before. My temp solution was to start a new RLE object

folivetti avatar Feb 28 '18 14:02 folivetti

While that may work, it has a larger overhead compared to a simple reset. Furthermore, you'll need to manually seed the environment after every reset in order to avoid using the same seed every episode.

nadavbh12 avatar Mar 07 '18 09:03 nadavbh12