search

pascalwhoop / tf_openailab_gpu_docker

39 stars 17 forks source link

Consider adding openai/baselines in the image? #3

Open zwfcrazy opened 6 years ago

zwfcrazy commented 6 years ago

Hi, I just want to thank you for the great work. I have been trying to combine tf with gym these days, but so far no success. I didn't even build the gym dockerfile successfully. This repo is really helpful. The VNC support is also very helpful.

As stated in the title, you may consider adding openai/baselines in the image as well.

Finally, just a quick note, you mentioned cudnn v7 in the README file but it is still cudnn v6 in the dockerfile.

pascalwhoop commented 6 years ago

Hi, I don't use the container anymore, although I might do so again in a few months. I am happy to go through PRs but can't invest time in it myself right now. There's also a lot of discussions about which cuDNN version is correct to use with which tensorflow version etc. So I think it would be worth making a solid docker container as a base for everyone. But unfortunately until I need it myself again, I won't have time to invest into that endeavour.