Maybe change `testing.fast_array_utils.Flags` design
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 25/100
Hướng nghiên cứu
Bắt đầu với phần thảo luận của pull request được liên kết và kiểm tra thiết kế hiện tại của testing.fast_array_utils.Flags, đặc biệt là cách xử lý Flags.Any và Flags(0). So sánh ba phương án được mô tả trong issue và làm rõ ngữ nghĩa dự kiến với các maintainer. Công việc được xem là hoàn tất khi đã thống nhất thiết kế flag và hành vi tương ứng cho mọi ArrayTypes.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Continued from https://github.com/scverse/fast-array-utils/pull/30#discussion_r1967609696 (cc @ilan-gold)
Currently we have Flags.Any, which is set for all ArrayTypes, in order to make sure that for each ArrayType “at”,
at.flags & ~Flags(0)is truthy, andnot (at.flags & Flags(0))is true.
This isn’t trivially the case, since each flag represents an attribute, and its absence the inverse attribute, so there is always one group that has no flags set. With the current set of flags (sparse, gpu, dask, disk), this group contains numpy.ndarray)
Alternatives:
- don’t use
Flags(0)as default forskip, butNoneinstead (unintuitive, one would expectskip=Flags(0)to skip nothing) - use a flag that is the inverse of another, e.g.
Flags.Dense(there is no longer one obvious way to do things, e.g. shall we skip sparse or select dense?) - switch the flags to
NoDaskandNoDisk, so thatFlags(0)represents a combination that can never exist (might be elegant if there were were words to express these concepts without negation)
- Ngôn ngữ chính
- Python
- Star
- 15
- Fork
- 5
- Merge trung bình
- 10 giờ 33 phút
- Pull request đã merge (30 ngày)
- 9
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 scverse/fast-array-utils
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 74/100
scverse/fast-array-utils#165 ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 30/100
scverse/fast-array-utils#215 · 1 bình luận ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 65/100
scverse/fast-array-utils#213 · 4 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
scverse/fast-array-utils#179 · 23 bình luận ·
-
type: numpy/scipy
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
scverse/fast-array-utils#128 ·
Tất cả issue của scverse/fast-array-utils
Issue tương tự
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 75/100
-
hcocena Đang mởpolicies-accepted pre-review precheck-passed
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
Bioconductor/BiocContributions#214 · 5 bình luận ·
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 92/100
TencentCloud/Octop#1169 · 1 bình luận ·
-
[开源推荐] 在老板拷问你之前,先让 AI 灵魂拷问你 Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
521xueweihan/HelloGitHub#3778 ·
-
The version checker's trailing attribute region has no control for a less-than inside a quoted value Đang mởarea: dashboard area: tests bug perceived difficulty: 2 python
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
Nitjsefnie-Harness-Commons/daedalus#1105 · 1 bình luận ·