Incorporate distributed RL framework, Ape-X and Ape-X DQN #246
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Accept current change (master) for PongNoFrameskip-v4 dqn config * Make env_info more explicit in run_pong script (accept incoming change) * Make learner return cpu state_dict (accept incoming change)
Implement Logger and test wandb functionality Add worker and logger render in argparse Implement load_param() method in logger and worker
…__init__ for all ApeX actors Implement test() in distributed architectures (load from checkpoint and run logger test())
* Move collect_data from worker class to ApeX Wrapper * Change hyperparameters around * Add worker-verbose as argparse flag
|
This pull request introduces 1 alert when merging 4342b78 into 815a1ca - view on LGTM.com new alerts:
|
|
This pull request introduces 1 alert when merging d36212a into 815a1ca - view on LGTM.com new alerts:
|
* Add separate integration test for ApeX * Add integration test flag to pong
|
This pull request introduces 1 alert when merging 1ab5e35 into 815a1ca - view on LGTM.com new alerts:
|
|
This pull request introduces 1 alert when merging b954f4d into 815a1ca - view on LGTM.com new alerts:
|
|
This pull request introduces 1 alert when merging a7be46b into 815a1ca - view on LGTM.com new alerts:
|
cyoon1729
changed the title
|
This pull request introduces 1 alert when merging 15bce10 into 815a1ca - view on LGTM.com new alerts:
|
|
This pull request introduces 1 alert when merging ec150b5 into 815a1ca - view on LGTM.com new alerts:
|
|
|
|
#247 is solved. |
MrSyee
approved these changes
Jun 23, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#245