aws / aws/aws-step-functions-data-science-sdk-python

Make sagemaker an optional dependency

Abierto
#184 5 comentarios 3 reacciones 0 asignados Ver en GitHub
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

Abrir la 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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.