Update Notebooks and README - #356
Conversation
7393b00 to
2c478a9
Compare
There was a problem hiding this comment.
Pull Request Overview
This PR enhances documentation and usability by adding interactive notebook badges and improving the quickstart experience. The changes make it easier for users to launch the Chronos-2 quickstart notebook in cloud environments and emphasize production deployment options.
- Added "Open in SageMaker Studio Lab" and "Open in Colab" badges to both the README and the quickstart notebook
- Removed the
-U(upgrade) flag from pip install command to prevent unintended package upgrades - Updated references to point to the new Chronos-2 SageMaker deployment tutorial
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| notebooks/chronos-2-quickstart.ipynb | Added interactive badges for SageMaker and Colab, removed -U flag from pip install, moved comment position, and updated Python version metadata |
| README.md | Added production deployment tip with SageMaker JumpStart, added interactive badges for the quickstart notebook, and updated SageMaker deployment tutorial reference |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| > [!TIP] | ||
| > For reliable production use, we recommend using Chronos-2 models through [Amazon SageMaker JumpStart](https://aws.amazon.com/sagemaker/ai/jumpstart/). Check out [this tutorial](notebooks/deploy-chronos-to-amazon-sagemaker.ipynb) to learn how to deploy Chronos-2 inference endpoints to AWS with just a few lines of code. | ||
|
|
||
|
|
There was a problem hiding this comment.
[nitpick] Excessive blank lines after the TIP block reduce readability. Remove one of the two consecutive blank lines at lines 65-66 to maintain consistent spacing in the documentation.
| "\n", | ||
| "\n", |
There was a problem hiding this comment.
[nitpick] Excessive blank lines after the badge links reduce readability. The three consecutive blank lines at lines 13-15 should be reduced to one or two blank lines for better visual consistency.
| "\n", | |
| "\n", |
*Issue #, if available:* *Description of changes:* By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice. --------- Co-authored-by: Oleksandr Shchur <oleks.shchur@gmail.com>
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.