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

Random seed in ALE cannot be set #254

Closed
katja-hofmann opened this issue Aug 3, 2016 · 1 comment
Closed

Random seed in ALE cannot be set #254

katja-hofmann opened this issue Aug 3, 2016 · 1 comment
Labels
Milestone

Comments

@katja-hofmann
Copy link
Member

The code in https://github.com/Microsoft/malmo/blob/63fbaad6501905a6667e393d4d6318a6107be7fe/Malmo/src/ALEAgentHost.cpp#L83 suggests that the same seed (123) is used on every run using the ALE environment. This should be set (a) to a different seed on each run, or (b) to a user-provided seed. Ideally, both should be possible.

@DaveyBiggers DaveyBiggers added this to the Dolphin milestone Aug 5, 2016
@DaveyBiggers DaveyBiggers added P3 P2 and removed P3 labels Aug 5, 2016
@DaveyBiggers
Copy link
Member

Have added ALEAgentHost::setSeed(int).
(Not ideal, since it breaks the whole keeping-the-agent-host-inferfaces-the-same paradigm, but then, that never made much sense anyway.)

Calling setSeed(123) will reproduce the previous behaviour, which was always setting the seed to 123.
Calling setSeed(0) will cause the ALE to seed from the current time each go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants