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.

Sign up for GitHub

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

Jump to bottom

feature(pu): add gfootball imitation learning and reinforcement learning demo config #352

Closed
wants to merge 23 commits into from
Closed

feature(pu): add gfootball imitation learning and reinforcement learning demo config #352

wants to merge 23 commits into from

Conversation

puyuan1996
Copy link
Collaborator

Description

  • adapt gfootball env to DI-engine env interface.
  • add gfootball imitation learning config, use the rule_based_bot and kaggle_5th_place model to collect demo data.
  • add episode_to_transitions_filter to only use the demo data whose return is larger than a given threshold.
  • add dqn config for gfootball.

Related Issue

#335

TODO

Check List

  • merge the latest version source branch/repo, and resolve all the conflicts
  • pass style check
  • pass all the tests

@PaParaZz1 PaParaZz1 changed the title feature(pu): add gfootball imitation learning and reinforcement learnning demo config feature(pu): add gfootball imitation learning and reinforcement learning demo config Jun 7, 2022
@PaParaZz1 PaParaZz1 added bug Something isn't working env Questions about RL environment labels Jun 7, 2022
@puyuan1996 puyuan1996 mentioned this pull request Jun 15, 2022
Closed
11 tasks
puyuan added 2 commits June 15, 2022 19:22
@puyuan1996
Copy link
Collaborator Author

we have a new PR #418.

@puyuan1996 puyuan1996 closed this Jul 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug Something isn't working env Questions about RL environment
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@puyuan1996 @PaParaZz1