-
Notifications
You must be signed in to change notification settings - Fork 3
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
Dev/pydantic #93
Merged
Merged
Dev/pydantic #93
Changes from 12 commits
Commits
Show all changes
13 commits
Select commit
Hold shift + click to select a range
d0d83d8
Enabling and using configuration files for saving and loading configu…
beardyFace bfac794
Not happy
beardyFace 9c3e1e0
Sorted parameter passing and external modifications
beardyFace 9fef7e0
Tidied up RLParser.py
beardyFace aa4c951
Updated README, plotter.py for new configurations
beardyFace 41b2aa2
automated configurations
beardyFace 70311c9
Updated README
beardyFace c495a2d
Updated tests
beardyFace f4eedcf
Removed unrequired rich import
beardyFace 631660a
fixed unit tests...maybe
beardyFace f531095
fixed unit tests...maybe
beardyFace 4ea4caa
ACtually actually fixed pytests
beardyFace 9fe6fa2
Fixed TD3 and README
beardyFace File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5,7 +5,6 @@ | |
import torch | ||
import torch.nn.functional as F | ||
|
||
|
||
class DDPG: | ||
|
||
def __init__(self, | ||
|
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -13,7 +13,6 @@ | |
import torch | ||
import torch.nn.functional as F | ||
|
||
|
||
class SAC: | ||
def __init__(self, | ||
actor_network, | ||
|
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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -9,7 +9,6 @@ | |
import torch | ||
import torch.nn.functional as F | ||
|
||
|
||
class TD3(object): | ||
def __init__(self, | ||
actor_network, | ||
|
@@ -109,15 +108,12 @@ def train_policy(self, experiences): | |
self.actor_net_optimiser.step() | ||
|
||
# Update target network params | ||
for target_param, param in zip(self.target_critic_net.Q1.parameters(), self.critic_net.Q1.parameters()): | ||
target_param.data.copy_(param.data * self.tau + target_param.data * (1.0 - self.tau)) | ||
|
||
for target_param, param in zip(self.target_critic_net.Q2.parameters(), self.critic_net.Q2.parameters()): | ||
for target_param, param in zip(self.target_critic_net.parameters(), self.critic_net.parameters()): | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Two targets nets. should be differents |
||
target_param.data.copy_(param.data * self.tau + target_param.data * (1.0 - self.tau)) | ||
|
||
for target_param, param in zip(self.target_actor_net.parameters(), self.actor_net.parameters()): | ||
target_param.data.copy_(param.data * self.tau + target_param.data * (1.0 - self.tau)) | ||
|
||
info['actor_loss'] = actor_loss | ||
|
||
# Building Dictionary | ||
|
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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,7 +4,6 @@ | |
import numpy as np | ||
import torch.nn.functional as F | ||
|
||
|
||
class DQN: | ||
|
||
def __init__(self, | ||
|
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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -9,7 +9,6 @@ | |
import numpy as np | ||
import torch.nn.functional as F | ||
|
||
|
||
class DoubleDQN: | ||
|
||
def __init__(self, | ||
|
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
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add 3, TD3