amplab / SparkNet

Distributed Neural Networks for Spark
MIT License
603 stars 172 forks source link

Adding more details to steps for ImageNet App in Readme #62

Closed rahulbhalerao001 closed 8 years ago

robertnishihara commented 8 years ago

Looks good! To make it a little simpler, can you replace steps 2 and 3 with a single step that says something like "Put the training and test data on S3 (see instructions in this issue)." And then renumber the subsequent steps appropriately.

rahulbhalerao001 commented 8 years ago

Ok, I have simplified the instructions and renumbered the subsequent steps.

robertnishihara commented 8 years ago

Looks excellent! Can you squash those commits together, and then I'll merge it! http://stackoverflow.com/questions/5189560/squash-my-last-x-commits-together-using-git

rahulbhalerao001 commented 8 years ago

I have squashed the commits together. Thank you for the link with the information.

robertnishihara commented 8 years ago

Thanks a lot!

robertnishihara commented 8 years ago

Addresses #59.