aws / aws/sagemaker-python-sdk

Make torch an optional dependency / extra

Offen
#5,531 5 Kommentare 7 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
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

Beitragsleitfaden öffnen

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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.