aws / aws/aws-step-functions-data-science-sdk-python
Make sagemaker an optional dependency
- Lenguaje dominante
- Python
- Estrellas
- 299
- Forks
- 84
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
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
Guía de contribución
Línea de trabajo
Empieza en setup.py e inspecciona install_requires y extras_require; después, consulta el README para ver las instrucciones de instalación. Verifica que la instalación predeterminada ya no requiera sagemaker y que los usuarios puedan instalar la dependencia opcional con el comando pip documentado; revisa el impacto en la versión mayor antes de cambiarlo.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- aws, python
- Área
- build-system, cloud, documentation, machine-learning
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Estado de actividad
- Estancado
- Claridad
- Bien especificado
- Aptitud para principiantes
- 45/100