Add get_processes() to wrap v1/system/processes
Đánh giá
Issue này chưa được đánh giá.
Mô tả
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.)
- Ngôn ngữ chính
- R
- Star
- 54
- Fork
- 27
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của posit-dev/connectapi
-
get_procs() @return docs don't match returned columns, and pid column is the row indexCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
posit-dev/connectapi#532 ·
-
Move uuid to SuggestsĐang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 62/100
posit-dev/connectapi#249 ·
-
get_procs() docs incorrectly say results are limited to the responding HA nodeCó thể đã có người làm @jmwoliver đã nhận hôm nay. Đang mở
posit-dev/connectapi#529 · 1 người được giao ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 65/100
posit-dev/connectapi#527 ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
posit-dev/connectapi#526 · 1 reaction ·
Tất cả issue của posit-dev/connectapi
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
Huber-group-EMBL/Rarr#294 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
-
Incorrect spanish translationĐang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 82/100
-
Windows source builds fail: cmdstanpy 1.3.0 downloads Rtools40 from a retired CRAN URL (HTTP 404)Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100