Add y/n option to missing libraries
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 45/100
Línea de trabajo
Comienza por el decorador @requires utilizado por cwms-cli y sigue cómo las bibliotecas faltantes producen actualmente el error. Comprueba cómo la CLI distingue las sesiones interactivas de usuarios humanos de los trabajos headless de CWMS Batch, y revisa sys.executable para conocer el comportamiento de la instalación. Se considera terminado cuando los usuarios interactivos pueden aceptar o rechazar la instalación, mientras que los trabajos headless conservan el error normal.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
CLI Script
cwms-cli
Feature Proposal
If a user does not have a required library, you will see this due to the @requires decorator.
cwms-cli blob list
Error: Missing module(s):
- `cwms` (install: `cwms-python`) — CWMS REST API Python client [docs](https://github.com/HydrologicEngineeringCenter/cwms-python)
Install missing packages:
pip install cwms-python
Should the CLI prompt the user y/n to auto install the library?
I.e. something like
Would you like to install missing packages?
pip install cwms-python
> [Y]es/[N]o: N
Bonus, we can use sys.executable to install it so that it will also respect the current python env.
Thinking we can build this into the @requires decorator
Care: If a headless job runs (CWMS Batch) we do not want any prompts to hang the job. They should instead throw the normal error. We want to make sure interactive prompts are reserved for human sessions.
- Lenguaje dominante
- Python
- Estrellas
- 3
- Forks
- 5
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la 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.
Más de HydrologicEngineeringCenter/cwms-cli
-
Suggested Clarification about Windows EnvironmentPosiblemente ocupada @krowvin la tomó hace 48 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
what is cwms cli?Posiblemente ocupada @krowvin la tomó hace 48 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
HydrologicEngineeringCenter/cwms-cli#284 · 3 comentarios · 1 reacción ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
enhancement
Dificultad 3/5 1-2 días Aptitud para principiantes 64/100
-
Read the Docs suggestion - "What is cwms-cli" - who should install it, and what can you do once its installedPosiblemente ocupada @krowvin la tomó hace 48 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
Todos los issues de HydrologicEngineeringCenter/cwms-cli
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
UKGovernmentBEIS/inspect_ai#6512 ·
Los mantenedores suelen responder en 1 día
-
area/c++ area/core kind/bug-report
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
tensorflow/quantum#1140 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
Kav-K/GPTDiscord#501 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
UKGovernmentBEIS/inspect_evals#2863 · 1 comentario ·
Los mantenedores suelen responder en 1 día