Skip to content
Please note that GitHub no longer supports your web browser.

We recommend upgrading to the latest Google Chrome or Firefox.

Learn more
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

Incorporate PySpark setup into overall documentation. #16

Merged
merged 3 commits into from Oct 28, 2019

Conversation

@ruebot
Copy link
Member

ruebot commented Oct 26, 2019

  • Removes standalone PySpark documentation.
  • Incorporates PySpark setup into getting started documentation.
  • Incorporates PySpark examples into overall documentation.
  • Breaks out scaling documentation to it's own documentation.
  • Removes cruft.
  • Renames files so they're all lowercase now.
  • Updates README ToC
- Removes standalone PySpark documentation.
- Incorporates PySpark setup into getting started documentation.
- Incorporates PySpark examples into overall documentation.
- Breaks out scaling documentation to it's own documentation.
- Removes cruft.
- Renames files so they're all lowercase now.
- Updates README ToC
@ruebot ruebot requested review from lintool and ianmilligan1 Oct 26, 2019
@ianmilligan1

This comment has been minimized.

Copy link
Member

ianmilligan1 commented Oct 26, 2019

Awesome- will review this afternoon hopefully!

Copy link
Member

ianmilligan1 left a comment

Some minor changes and suggestions, but otherwise this looks great. Thanks @ruebot !

current/README.md Outdated Show resolved Hide resolved
current/README.md Outdated Show resolved Hide resolved
current/README.md Outdated Show resolved Hide resolved
current/toolkit-lesson.md Outdated Show resolved Hide resolved
@ruebot ruebot force-pushed the torch-pyspark-only-page branch from c91dca3 to 3a42fac Oct 26, 2019
Copy link
Member

ianmilligan1 left a comment

Looks great! Happy to merge once @lintool gives a thumbs up? (Or he can merge)

@ianmilligan1 ianmilligan1 merged commit 4cc429a into master Oct 28, 2019
@ianmilligan1 ianmilligan1 deleted the torch-pyspark-only-page branch Oct 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.