aws / aws/aws-step-functions-data-science-sdk-python
Make sagemaker an optional dependency
- Lingua principale
- Python
- Stelle
- 299
- Fork
- 84
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
I'd like to see sagemaker marked as an optional dependency, as it is not always required.
### Use Case
Not all users of this package are using the sagemaker dependency, and it adds a lot of bloat to overall installations by forcing users to have gcc and g++ to be able to install numpy and pandas.
### Proposed Solution
It's quite simple to actually do this.
In `setup.py` you'd have to change `install_requires` and add `extras_require`:
```python
setup(
# [...]
install_requires = [
"boto3>=1.14.38",
"pyyaml"
]
extras_require = {
"sagemaker": ["sagemaker>=2.1.0"],
"test": [] # [...]
}
# [...]
)
```
In addition to this, some extra documentation should probably be added to the README to make it easier for users to understand how to install the extra dependency (`pip install stepfunctions[sagemaker]`).
### Other
If this is implemented, the major version probably has to be bumped because it's a change in how the package is installed.
---
* [x] :wave: I may be able to implement this feature request
* [x] :warning: This feature might incur a breaking change
---
This is a :rocket: Feature Request
Guida per i contributori
Apri la guida per i contributori
Direzione di ricerca
Inizia da setup.py e ispeziona install_requires ed extras_require, quindi controlla il README per le istruzioni di installazione. Verifica che l’installazione predefinita non richieda più sagemaker e che gli utenti possano installare la dipendenza opzionale con il comando pip documentato; esamina l’impatto sulla versione principale prima di modificarla.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- aws, python
- Ambito
- build-system, cloud, documentation, machine-learning
- Tipo di issue
- Funzionalità
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Stato di attività
- Ferma
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 45/100