-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
Clean up RTD Website #1 #3312
Clean up RTD Website #1 #3312
Conversation
…_fasttext/blazingtext_hosting_pretrained_fasttext.ipynb
…your_own/Batch_Transform_BYO_XGB.ipynb
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
…ne_learning_model_ground_truth_demo_image_classification/from_unlabeled_data_to_deployed_machine_learning_model_ground_truth_demo_image_classification.ipynb
…3D-point-cloud-input-data-processing.ipynb
…l/object_detection_tutorial.ipynb
…eling_tutorial.ipynb
…beling_workflows_with_active_learning/bring_your_own_model_for_sagemaker_labeling_workflows_with_active_learning.ipynb
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
…3D-point-cloud-input-data-processing.ipynb
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
…k_image_datasets/image-classification-with-shutterstock-datasets.ipynb
…uto_insurance_claim_processing.ipynb
…ace_safety/improving_industrial_workplace_safety.ipynb
…eneric_sample_notebook_to_perform_inference_on_ML_model_packages_from_AWS_Marketplace.ipynb
…xt-generation/creative-writing-using-gpt-2-text-generation.ipynb
…s_marketplace_ml_models/amazon_augmented_ai_with_aws_marketplace_ml_models.ipynb
…l/using-image-classification-models.ipynb
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
…extending_our_containers.ipynb
…with_your_own_model/tensorflow_serving_pretrained_model_elastic_inference.ipynb
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
@@ -485,7 +485,7 @@ | |||
"cell_type": "markdown", | |||
"metadata": {}, | |||
"source": [ | |||
"#### Inspect the Batch Transform Output in S3" | |||
"##### Inspect the Batch Transform Output in S3" |
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.
Does 5 headers deep look okay on GitHub and RTD?
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.
It looks fine to me at https://atqy-amazon-sagemaker-examples.readthedocs.io/en/atqy-correct-rtd-rst/aws_marketplace/index.html (it is the first notebook listed). I generally don't agree with that many level of headers but am just following what the author initially wrote. Let me know if you think it is not fine though.
.. toctree:: | ||
:maxdepth: 1 | ||
|
||
../introduction_to_applying_machine_learning/linear_time_series_forecast/linear_time_series_forecast |
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.
Is there any other Linear Learner notebook we can highlight if this one was deleted?
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.
Unfortunately, I have searched and not found one. There are many linear learner examples but not one with time series application.
Issue #, if available:
Description of changes:
There are two types of errors that this PR cleans up:
(A) Any error in the RST files (i.e. empty sections, broken links, etc)
(B) Reformatting Markdown headers in notebooks so that they are displayed correctly in RTD website
Testing done:
Looked at https://atqy-amazon-sagemaker-examples.readthedocs.io/en/atqy-correct-rtd-rst/ to see if notebooks are formatted correctly. The pages affected are:
https://atqy-amazon-sagemaker-examples.readthedocs.io/en/atqy-correct-rtd-rst/get_started/index.html
https://atqy-amazon-sagemaker-examples.readthedocs.io/en/atqy-correct-rtd-rst/aws_sagemaker_studio/index.html
https://atqy-amazon-sagemaker-examples.readthedocs.io/en/atqy-correct-rtd-rst/autopilot/index.html
https://atqy-amazon-sagemaker-examples.readthedocs.io/en/atqy-correct-rtd-rst/sagemaker-experiments/index.html
https://atqy-amazon-sagemaker-examples.readthedocs.io/en/atqy-correct-rtd-rst/label_data/index.html
https://atqy-amazon-sagemaker-examples.readthedocs.io/en/atqy-correct-rtd-rst/aws_marketplace/index.html
https://atqy-amazon-sagemaker-examples.readthedocs.io/en/atqy-correct-rtd-rst/training/bring_your_own_container.html
https://atqy-amazon-sagemaker-examples.readthedocs.io/en/atqy-correct-rtd-rst/inference/bring_your_own_container.html
Merge Checklist
Put an
x
in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.tox -e black-format,black-nb-format
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.