-
Notifications
You must be signed in to change notification settings - Fork 32
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
image size #6
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi! @denisyarats
It's been a long time since this repo was published. Still hoping you'll respond to this issue.
Have you ever tried this method with a larger image size? In your code, the height and width of 84 is the used as the biggest size.
I tried to increase the size to 128 but the memory cost reached almost 100GB.
For the tested
dm_control
tasks, the image size of 84 seems to be adequate, but may not be enough for other tasks.Would you mind sharing some of your opinions? Thanks a lot!
The text was updated successfully, but these errors were encountered: