Add get_processes() to wrap v1/system/processes
Evaluación
Este issue todavía no se ha evaluado.
Descripción
I think we want a new function name than get_procs(), the old one. The new function should call GET v1/system/processes with Accept: text/csv to get a CSV download and then reading the data in should be straightforward. If that 404s, then we're pointing at old Connect, so call the old API (justget_procs()), and map the old column names to the new ones. We have a few of these "call the new API and if it fails call the old one" setups in the package already.
The new API supports a few filtering params that we could add as arguments to get_processes()
get_procs() should call .Deprecated() (or if there is a fancier lifecycle:: function we can use that since we already depend on it) and tell people to use get_processes() instead. (Though we wouldn't want to raise the warning if we're calling the logic from within get_processes() as the fallback, so either suppress the warning there or factor out the logic that does the requests and formatting.)
- Lenguaje dominante
- R
- Estrellas
- 54
- Forks
- 27
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una 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 posit-dev/connectapi
-
get_procs() @return docs don't match returned columns, and pid column is the row indexPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
posit-dev/connectapi#532 ·
-
Move uuid to SuggestsAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 62/100
posit-dev/connectapi#249 ·
-
get_procs() docs incorrectly say results are limited to the responding HA nodePosiblemente ocupada @jmwoliver la tomó hoy. Abierto
posit-dev/connectapi#529 · 1 asignado ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 65/100
posit-dev/connectapi#527 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
posit-dev/connectapi#526 · 1 reacción ·
Todos los issues de posit-dev/connectapi
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Huber-group-EMBL/Rarr#294 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
Incorrect spanish translationAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
-
Windows source builds fail: cmdstanpy 1.3.0 downloads Rtools40 from a retired CRAN URL (HTTP 404)Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100