Skip to content

Pin sagemaker SDK <3.0.0 across all sagemaker notebooks#615

Open
Sithembiso-Mashinini wants to merge 1 commit intohuggingface:mainfrom
Sithembiso-Mashinini:pin-sagemaker-sdk-below-v3
Open

Pin sagemaker SDK <3.0.0 across all sagemaker notebooks#615
Sithembiso-Mashinini wants to merge 1 commit intohuggingface:mainfrom
Sithembiso-Mashinini:pin-sagemaker-sdk-below-v3

Conversation

@Sithembiso-Mashinini
Copy link

Summary

  • Pin sagemaker<3.0.0 in all 31 sagemaker notebook pip install lines to prevent SDK v3 from being installed via --upgrade
  • SDK v3 removes HuggingFace estimator, HuggingFaceModel, Predictor, and other APIs used across these notebooks
  • HuggingFace has not yet published v3-compatible patterns — their docs state all tutorials still use SDK v2

Changes

  • Notebooks with existing version constraints (e.g. "sagemaker>=2.140.0") get an upper bound added: "sagemaker>=2.140.0,<3.0.0"
  • Notebooks with bare sagemaker get pinned to "sagemaker>=2.0.0,<3.0.0"
  • Notebooks with exact pins (e.g. "sagemaker==2.163.0") are relaxed to "sagemaker>=2.163.0,<3.0.0"

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

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.

1 participant