-
Notifications
You must be signed in to change notification settings - Fork 46
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
Documentation for FindTheGoal #55
Comments
@rocanaan : You are right, there was a small issue. While I fixed it, I realised the docstirngs for FindTheGoal are not written. @idvhfd confirmed that he will be adding them soon. And wrt your questions,
@idvhfd : This is what needs a touch up : https://github.com/crowdAI/marLo/blob/master/marlo/envs/FindTheGoal/main.py#L12 |
@spMohanty How do you configure the env to have |
@spMohanty what I mean is that when I run the environment I never see any block other than the gold one. I looked at the mission file and didn't find a parameter to set that up. If during evaluation our agent can encounter non gold block, how are we supposed to set our local environment to reproduce this diversity? |
Apologies for the delay, I was unable to attend this earlier. I added a brief description in the same style as the previous ones (awaiting confirm and pull). Thanks for pointing this out and please do accept my apologies for creating this situation. Let me know if anything else like this is needed. @justinglibert the seed is being passed as a parameter, that's where the block comes from. Waiting on @spMohanty for exact path to that bit as I can't look for it right now. |
Hello,
I was looking at issue #32 and the documentation for the environments and realized the FindTheGoal environment is one of the few environments with no documentation.
Since this is the environment that will be used for the first phase, I would like to know if there is an ETA for the documentation on this environment to be made available.
If I may make some suggestions, I would like to know:
Thank you very much!
The text was updated successfully, but these errors were encountered: