Tech debt: consolidate test data from #187 and move signifDigitsEpsilon to veupathUtils
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 35/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Cần làm rõ
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- r
- Lĩnh vực
- testing-qa, tooling
Hướng nghiên cứu
Start with the discussion and review on pull request #187, then inspect the existing test object around line 25, including testDF and the tac data. Trace signifDigitEpsilon and the veupathUtils integration, noting that the function is not currently exported from plot.data. Done means consolidating the test data and relocating the utility without losing the behavior described in the referenced review.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
@asizemore suggested here
is there a reasonable way we can modify the existing test object (like in line 25) to fail in the same way? Ideally we've been trying to keep the test data to a minimum set that can be used or manipulated for nearly all the tests. Or if we could pick out 500 values of the tac data and add them as a new column contE or something like that in testDF that would similarly prevent test data object sprawl, if possible.
also see this regarding the function
(note that the signifDigitEpsilon is not currently exported from plot.data)
- Ngôn ngữ chính
- R
- Star
- 1
- Fork
- 0
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
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 VEuPathDB/plot.data
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 25/100
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
-
empty two by two Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
Tất cả issue của VEuPathDB/plot.data
Issue tương tự
-
bug triage_needed
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
pharmaverse/rtables#1123 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
Incorrect coaches in games.csv Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
-
Trying to reassign a reactiveVal to the name of an existing reactiveVal doesn't overwrite the latter Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100