deep-rl-class icon indicating copy to clipboard operation
deep-rl-class copied to clipboard

Published 20 hours ago • verified publisher icon huggingface

Clear up some unneeded and confusing parts of Unit 2

Open chaselambda opened this issue 3 years ago • 2 comments
trafficstars

I've added comments to why I think each of these changes are useful. Thanks!

chaselambda avatar Aug 08 '22 00:08 chaselambda

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

review-notebook-app[bot] avatar Aug 08 '22 00:08 review-notebook-app[bot]

Thanks for the update. Before merging it we need to remove epsilon from Step 4: Create a model dictionary 💾 and publish our trained model on the Hub 🔥

simoninithomas avatar Aug 11 '22 07:08 simoninithomas

done, thanks!

chaselambda avatar Aug 16 '22 02:08 chaselambda

Awesome thanks I merge it 🤗

simoninithomas avatar Aug 16 '22 06:08 simoninithomas