Add get_processes() to wrap v1/system/processes
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
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.)
- Lingua principale
- R
- Stelle
- 54
- Fork
- 27
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di posit-dev/connectapi
-
get_procs() @return docs don't match returned columns, and pid column is the row indexForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
posit-dev/connectapi#532 ·
-
Move uuid to SuggestsAperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 62/100
posit-dev/connectapi#249 ·
-
get_procs() docs incorrectly say results are limited to the responding HA nodeForse già presa @jmwoliver l’ha presa oggi. Aperta
posit-dev/connectapi#529 · 1 assegnatario ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 65/100
posit-dev/connectapi#527 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
posit-dev/connectapi#526 · 1 reazione ·
Tutte le issue di posit-dev/connectapi
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
Huber-group-EMBL/Rarr#294 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 82/100
-
Windows source builds fail: cmdstanpy 1.3.0 downloads Rtools40 from a retired CRAN URL (HTTP 404)Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100