Add newtype wrappers to fix scale/rate confusion
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 bằng cách xem xét các API phân phối hiện có, đặc biệt là phân phối mũ, để xác định những nơi mà các bên gọi hiện đang truyền các giá trị scale hoặc rate. So sánh thiết kế Rate, Scale và RateOrScale được đề xuất với các API đó; công việc hoàn tất khi sự phân biệt giữa các tham số và việc chấp nhận một trong hai dạng được xác định nhất quán trên tất cả các hàm bị ảnh hưởng.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
We have several functions which could take either scale or rate parameter. For example exponential distribution. I can never remember which should be used. I think we shold use type system to encode which kind of parameter should be used. It's also conveninent to be able to pass either rate or scale parameter. So I propose following design:
newtype Rate = Rate Double
newtype Scale = Scale Double
class RateOrScale a where
toRateParameter :: a → Double
toScaleParameter :: a → Double
Name of type class is hmm... not ideal.
- Ngôn ngữ chính
- Haskell
- Star
- 309
- Fork
- 77
- 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
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
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 haskell/statistics
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
haskell/statistics#235 ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
haskell/statistics#237 ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
haskell/statistics#236 ·
-
Bug: LinearTransform shiftĐang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 58/100
haskell/statistics#234 ·
-
Test failure Pearson correlationĐang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 42/100
haskell/statistics#231 · 3 bình luận ·
Tất cả issue của haskell/statistics
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
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 68/100
-
pkg:crucible-mir type:bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Heroes in statisticsĐang mở
Độ 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 78/100
GaloisInc/llvm-pretty#211 ·