sassoftware / sassoftware/python-sasctl
Older examples needs to be updated for JSONFiles API changes
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 52
- Forks
- 45
- Merge medio
- 23 h 38 min
- PR fusionados (30 d)
- 2
Descripción
Describe the issue
This example has some API calls that don't work with the updated version of sasctl - changes to JSONFiles:
https://github.com/sassoftware/python-sasctl/blob/master/examples/pzmmCompleteModelExampleMLFlowSklearn.ipynb
For example, writeModelPropertiesJSON is now write_model_properties_json and has many changes in the parameters.
Other examples:
writeFileMetadataJSON
writeVarJSON
I checked other examples and it seems any of them 6 months or older are now out of date with the newer sasctl API.
To Reproduce
Run this command using latest sasctl
J.writeVarJSON(inputsDict, isInput=True, jPath=zipFolder)
and you will get things like
AttributeError: 'JSONFiles' object has no attribute 'writeVarJSON'. Did you mean: 'write_var_json'?
Version
The SHA is 729e72c8785065109ccfd1ffdb9816d4bf1dce55
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza con examples/pzmmCompleteModelExampleMLFlowSklearn.ipynb y compara sus llamadas a JSONFiles con la API actual de sasctl. Busca writeModelPropertiesJSON, writeFileMetadataJSON y writeVarJSON en los demás ejemplos y, después, actualiza las llamadas y los parámetros obsoletos. La tarea estará terminada cuando los ejemplos antiguos se ejecuten con la última versión de sasctl sin el AttributeError informado.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- documentation
- Tipo de issue
- Documentación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100