Skip to content

Commit

Permalink
Fix creation of offline data to use terminated rather than done
Browse files Browse the repository at this point in the history
Summary: While recently the field `done` has been replaced by `terminated` almost everywhere, it incorrectly remained in `create_offline_data`, so we fix it here.

Reviewed By: yiwan-rl

Differential Revision: D55754147

fbshipit-source-id: 0b255f915c80b0f335ab6f122081805ab00d3b2a
  • Loading branch information
rodrigodesalvobraz authored and facebook-github-bot committed Apr 4, 2024
1 parent 4234ceb commit 575ca88
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ def create_offline_data(
"next_observation": action_result.observation,
"curr_available_actions": env.action_space,
"next_available_actions": env.action_space,
"done": action_result.done,
"terminated": action_result.terminated,
}

observation = action_result.observation
Expand Down

0 comments on commit 575ca88

Please sign in to comment.