Skip to content

Conversation

jsmul
Copy link
Contributor

@jsmul jsmul commented Jul 26, 2024

Issue #, if available:

Description of changes:

Implemented new file sctructure.

Testing done:

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.

  • I have verified that my PR does not contain any new notebook/s which demonstrate a SageMaker functionality already showcased by another existing notebook in the repository.
  • I have read the CONTRIBUTING doc and adhered to the example notebook best practices
  • I have updated any necessary documentation, including READMEs
  • I have tested my notebook(s) and ensured it runs end-to-end
  • I have linted my notebook(s) and code using black-nb -l 100 {path}/{notebook-name}.ipynb

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

jsmul and others added 30 commits June 13, 2024 12:31
aws#4678)

* Update SMP v2 notebooks to use latest PT2.3.1-TSM2.4.0 release.

* Update SMP v2 shared_scripts

* Update minimum sagemaker pysdk version to 2.224
* tutorials-after-initial-feedback

Added descriptive text to make the notebooks stand on their own.

* move athena notebook into dedicated folder

* renamed athena end2end notebooks

* moved pyspark notebook into dedicated directory

* minor change: consistent directory naming convention

* Added overview, headers, and explantory text

Tested the notebook end to end. Added more context for processing jobs and cleaning up. The output is visible in the cells.

* Added overview, headers, explanatory text

Also added troubleshooting note from further testing.

* fix directory locations for new notebooks

* clear notebook outputs

* added integration for ci test results

* updated formatting with black-nb

* update athena notebook: fix parse predictions

* fixed ci integration for pyspark-etl-training notebook

---------

Co-authored-by: Janosch Woschitz <[email protected]>
…xample notebooks (aws#4691)

* Archived remaining geospatial example notebooks

* Removed geospatial from README.md
* Add SageMaker MLflow examples

* Add badges

* Add MLflow setup notebook; upgrade SageMaker Python SDK for deployment notebook

* Linting

* More linting changes

---------

Co-authored-by: Bobby Lindsey <[email protected]>
…e tuning" (https://sim.amazon.com/issues/ML-16440) (aws#4657)

* initial commit of using step decorator for bedrock fine tuning

* ran black command on the notebook

* Added CI badges

* Added CI badges

* fixed typo in notebook title

---------

Co-authored-by: Ashish Rawat <[email protected]>
Co-authored-by: Zhaoqi <[email protected]>
* Deleted 17 duplicate notebooks (aws#4685)

* Updated README, removed broken links and fixed markdown (aws#4687)

* New Folder Structure Implementation - Archived remaining geospatial example notebooks (aws#4691)

* Archived remaining geospatial example notebooks

* Removed geospatial from README.md

* Archived remaining workshop notebooks (aws#4692)

* Archived outdated example notebooks between 1-90 views (aws#4693)

---------

Co-authored-by: jsmul <[email protected]>
This reverts commit 970d88e due to broken blog links
* adding notebook for forecast to canvas workshop

* formatting the notebook using black
Deleted 17 duplicate notebooks
jsmul added 21 commits July 23, 2024 10:47
Archived outdated example notebooks between 1-90 views
archived 17 notebookswith outdated/redundant functionality
archived 20 notebooks with outdated/redundant functionality
archived 20 notebooks with outdated/redundant functionality
archived 20 notebooks with outdated/redundant funtionality
archived 19 notebooks with outdated/redundant functionality
@jsmul jsmul requested a review from a team as a code owner July 26, 2024 21:20
@jsmul jsmul requested review from Aditi2424 and removed request for a team July 26, 2024 21:20
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@sagemaker-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: SageMakerNotebooksProd-USWEST2-amazon-sagemaker-examples-pr
  • Commit ID: baab858
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: SageMakerNotebooksProd-USWEST2-sagemaker-examples-code-formatting
  • Commit ID: baab858
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: SageMakerNotebooksProd-USWEST2-sagemaker-examples-link-check
  • Commit ID: baab858
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@zhaoqizqwang zhaoqizqwang merged commit 1732bc2 into aws:proposed-folder-directory Jul 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants