check `dat` argument (and others) at the start of save_all_plots()
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 68/100
Hướng nghiên cứu
Tìm save_all_plots() và kiểm tra cách hàm này sử dụng dat và cấu trúc đầu ra từ convert_output(). Thêm bước xác thực sớm cho cấu trúc dự kiến, sau đó kiểm tra để đảm bảo đầu vào không hợp lệ sẽ nhanh chóng gây lỗi kèm thông báo rõ ràng, thay vì tạo ra nhiều thông báo ... failed to run..
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
I accidentally passed an object to the dat input of save_all_plots() that wasn't the output from convert_output() and I got 100+ lines of messages about ... failed to run. Adding a check at the start to confirm that the dat input is of the appropriate structure would allow the function to fail faster and more informatively.
- Ngôn ngữ chính
- R
- Star
- 20
- Fork
- 10
- Merge trung bình
- 3 ngày 9 giờ
- Pull request đã merge (30 ngày)
- 21
Chuẩn bị môi trường
Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.
- Không có Dockerfile hay tệp Docker Compose
- Không 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 nmfs-ost/stockplotr
-
[bug] plot_spawning_potential_ratio() example throwing errorCó thể đã có người làm @JZahner1 đã nhận 1 ngày trước. Đang mở
nmfs-ost/stockplotr#444 · 4 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Release Checklist - Monthly UpdateCó thể đã có người làm @Schiano-NOAA đã nhận 1 ngày trước. Đang mở
nmfs-ost/stockplotr#442 · 2 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
update SS3 testsCó thể đã có người làm @Schiano-NOAA đã nhận 10 ngày trước. Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
nmfs-ost/stockplotr#441 · 4 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Brainstorm ways to automatically update `save_all_plots()`Có thể đã có người làm @sbreitbart-NOAA đã nhận 9 ngày trước. Đang mở
nmfs-ost/stockplotr#432 · 4 bình luận · 2 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 15/100
nmfs-ost/stockplotr#422 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của nmfs-ost/stockplotr
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 71/100
-
Độ 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 76/100
OHDSI/PatientLevelPrediction#688 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
rfordatascience/tidytuesday#1098 ·