Skip to content

Better Multiagent implementation#3

Merged
sidzekrom merged 13 commits intomasterfrom
multi-agent
Jul 21, 2017
Merged

Better Multiagent implementation#3
sidzekrom merged 13 commits intomasterfrom
multi-agent

Conversation

@liamuk
Copy link
Copy Markdown
Collaborator

@liamuk liamuk commented Apr 30, 2017

Should train faster, allow more general nn architectures

src/rl/dqn.py Outdated
BATCH_SIZE = 32
MEMORY_SIZE = 10000
REPORT_INTERVAL = 1000
REPORT_INTERVAL = 10
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

somehow this got fucked up by a merge somewhere, should be higher

src/rl/dqn.py Outdated
(b_s, b_a, b_ns, b_r, b_t) = \
process_samples(batch, self.n_actions)
b_y = self._calc_q_update(b_ns, b_r, b_t)
print(b_y)
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this print was removed and somehow readded. please remove

@liamuk
Copy link
Copy Markdown
Collaborator Author

liamuk commented May 1, 2017

This PR is pretty fucked. @sidzekrom could you make a new branch based off of master with your one function addition and PR that?

@sidzekrom sidzekrom merged commit bcd5c33 into master Jul 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants