-
Notifications
You must be signed in to change notification settings - Fork 132
Docs cleanup #528
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
Docs cleanup #528
Conversation
Codecov Report
@@ Coverage Diff @@
## main #528 +/- ##
=======================================
Coverage 94.75% 94.75%
=======================================
Files 65 65
Lines 3949 3949
=======================================
Hits 3742 3742
Misses 207 207
Continue to review full report at Codecov.
|
@priyaramani has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM thanks for the fixes!
There is a failing doctest FYI https://github.com/pytorch/torchx/runs/6942958011?check_suite_focus=true |
@priyaramani has updated the pull request. You must reimport the pull request before landing. |
@priyaramani has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Update docs to reflect current verbiage/outputs for builtins, default scheduler and app handles. Also removes some unnecessary arguments in examples and fixes some typos.
Test:
cd docs && make html
(with airflow.md commented out)