-
Notifications
You must be signed in to change notification settings - Fork 153
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
event_infos tag bug #78
Comments
micahcarroll
moved this from Todo: low priority
to Todo: high priority
in Overcooked Improvements
Jun 10, 2022
micahcarroll
moved this from Todo: high priority
to Todo: low priority
in Overcooked Improvements
Jun 10, 2022
This issue is stale because it has been open for 30 days with no activity. It will be closed in 14 days unless there is some new activity |
This issue is stale because it has been open for 30 days with no activity. It will be closed in 14 days unless there is some new activity |
micahcarroll
moved this from Todo: low priority
to Todo: high priority overcooked_ai / demo
in Overcooked Improvements
Mar 29, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Reported via email:
I’m looking to use the ‘event_infos’ tags as propositional labels for my particular application, but it seems that sometimes these tags don’t get correctly updated.
For example, in the following (state, action, new_state) triple:
X X P X X
O →1 T
Xo →0 X
X D X S X
(Action.INTERACT, Action.INTERACT)
X X P X X
O →1t T
Xo →0 X
X D X S X
I can see, by inspecting the relevant variables, that
state.players[1].held_object
has valueNone
andnew_state.players[1].held_object
has valuetomato@(3, 1)
, but that if I look at:_, info = env.mdp.get_state_transition(state, (Action.INTERACT, Action.INTERACT,), False, env.mp)
Then the value of
info['event_infos']['tomato_pickup’]
is[False,False]
, when I believe it should be[False,True]
. This may well happen with other info tags as well, but I haven’t dug into that yet. Do you have any idea what might be causing this problem and/or how to fix it?For the record, I’m using whatever version of the repo is installed when running
pip install overcooked-ai
.The text was updated successfully, but these errors were encountered: