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.

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

desired_samples on line 593 of NeuralQLearner.lua not defined #1

Open
shehroze37-zz opened this issue Jun 24, 2016 · 3 comments
Open

Comments

@shehroze37-zz
Copy link

desired_samples is not defined and this gives an error. I guess this can't be self.num_samples because even with that it throws an error in sample() function (TransitionTable ) because the batchSize is greater than the bufferSize.

Cheers

@SimsGautam
Copy link
Collaborator

desired_samples should be self.num_samples -- good catch! What are the batchSize and bufferSize you are using? Did you try increasing the bufferSize?

@SimsGautam
Copy link
Collaborator

Hey @shehroze37, I just pushed a short fix to this. Let me know if you're still running into the same problem.

@shehroze37-zz
Copy link
Author

Sure I will

Cheers !

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

No branches or pull requests

2 participants