[bug]: plot_abundance_at_age allows selection of non age-based modules
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ó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 40/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- r
- Lĩnh vực
- data-visualization
Hướng nghiên cứu
Start with plot_abundance_at_age and the filter_data step it calls, and see how the module is picked from the shared "abundance" label. Reproduce the error by selecting NUMBERS_AT_LENGTH or SPR_SERIES. Done means the function only offers modules that work with age-based plots, or it fails with a clear message. The maintainer has not chosen a fix, so confirm the approach in the thread before coding.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
The plot_abundance_at_age function currently identifies data based on the "abundance" label. However, many models reuse this label across modules, including "NUMBERS_AT_AGE", "NUMBERS_AT_LENGTH", "SPR_SERIES", "TIME_SERIES", etc. The default behavior of filter_data allows for user selection of which module to use for plotting, but some of these modules will throw errors if selected (e.g., "NUMBERS_AT_LENGTH").
This seems like it could be readily confusing for many users. It would probably be a good idea to restrict the allowable modules for users to select to only those that are applicable to the plotting function being used ("NUMBERS_AT_AGE" in this case).
This issue likely applies to more cases than the one identified here, and I am not confident as to a solution, but wanted to point it out.
- Ngôn ngữ chính
- R
- Star
- 20
- Fork
- 9
- Merge trung bình
- 3 ngày 10 giờ
- Pull request đã merge (30 ngày)
- 15
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
-
Brainstorm ways to automatically update `save_all_plots()`Có thể đã có người làm @sbreitbart-NOAA đã nhận hôm nay. Đang mở
nmfs-ost/stockplotr#432 · 1 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
-
[fix]: module name selection retained in environment when `make_rda = T`Có thể đã có người làm @sbreitbart-NOAA đã nhận 5 ngày trước. Đang mởbug
nmfs-ost/stockplotr#421 · 5 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
[feat]: Create function to help user directly add alt text and caption to csv for external objectsCó thể đã có người làm @sbreitbart-NOAA đã nhận 7 ngày trước. Đang mở
nmfs-ost/stockplotr#412 · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 18/100
nmfs-ost/stockplotr#405 · 1 reaction ·
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ự
-
BC Aquarius drops end_dateĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
HakaiInstitute/hydrocan#35 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
core
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 83/100
sims1253/ry#720 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 64/100
OHDSI/PhenotypeR#710 ·