aws / aws/sagemaker-python-sdk
Make torch an optional dependency / extra
- Vorherrschende Sprache
- Python
- Sterne
- 2.3k
- Forks
- 1.3k
- Ø Merge
- 1 T. 22 Std.
- Gemergte PRs (30 T.)
- 35
Beschreibung
Modern packages use optional dependecies / extras for specific functionality. I assume this is very well possible for the sagemaker sdk.
A similar request was posted in discussions: https://github.com/aws/sagemaker-python-sdk/discussions/5441
I couldn't find any actual "issue" and hope this gets more visibility.
The main and most urgent topic is the dependency of pytorch which requires 354/620mb size in a clean venv. I would assume that most use cases don't need pytorch at all.
Beitragsleitfaden
Rechercherichtung
Im Issue werden keine Dateien oder Tests genannt. Beginne damit, die Dependency-Deklarationen und die Installations-/Extras-Konfiguration des SDK zu lokalisieren, und verfolge dann, wo PyTorch importiert oder vorausgesetzt wird; vergleiche eine saubere Installation mit dem vorgeschlagenen Extra. Als erledigt gilt, dass sich das Basispaket ohne PyTorch installieren lässt, während PyTorch-abhängige SageMaker-Funktionalität über ein explizites Extra weiterhin nutzbar bleibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- aws, python, pytorch
- Bereich
- build-system, machine-learning
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Ruhig
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 50/100