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

"none"/"air" for equipped_items (observation)? #164

Closed
keisuke-nakata opened this issue Jul 23, 2019 · 3 comments

Comments

@keisuke-nakata
Copy link
Collaborator

commented Jul 23, 2019

equipped_items (which is part of observation) has options 'none', 'air'.
I am confused from the difference between among these options.

I can understand the "equip" none, air for action as documented, but for observation, what do "equipped_items = none/air" mean?

@keisuke-nakata keisuke-nakata changed the title "none"/ "none"/"air" for equipped_items (observation) and equip (action) Jul 23, 2019
@keisuke-nakata keisuke-nakata changed the title "none"/"air" for equipped_items (observation) and equip (action) "none"/"air" for equipped_items (observation)? Jul 23, 2019
@brandonhoughton

This comment has been minimized.

Copy link
Collaborator

commented Jul 23, 2019

equip none is a no-op action, it does not affect the state of the agent
equip air attempts to equip the item air which effectively unequips any item the agent is holding

Note that if the agent does not have any space in their inventory (and as such there are no free slots) equip air will not have any effect. This is a rare case however and we have not encountered this for agents in the competition.

@keisuke-nakata

This comment has been minimized.

Copy link
Collaborator Author

commented Jul 31, 2019

What is the difference between observation.equipped_items.mainhand none and observation.equipped_items.mainhand air ?

@MadcowD

This comment has been minimized.

Copy link
Collaborator

commented Aug 1, 2019

Well I suspect that if the user is equipping something not in the observation space it will be none and if there is nothing in the hand then it will be air. Perhaps we should make these one and the same.

@MadcowD MadcowD closed this Sep 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.