Add newtype wrappers to fix scale/rate confusion
还没有人认领这个 Issue。
评估
调研方向
首先检查现有的分布 API，尤其是指数分布，以确定调用方当前在哪些地方传递 scale 或 rate 值。将提议的 Rate、Scale 和 RateOrScale 设计与这些 API 进行比较；当受影响的各个函数都一致地定义了参数区别以及接受任一形式时，这项工作就完成了。
由索引模型根据 Issue 内容生成。
描述
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.
- 主要语言
- Haskell
- 星标
- 309
- 派生
- 77
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
这个项目没有提供开发容器、Dockerfile 或贡献指南，环境需要你自己搭建：先看它的 README，通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
haskell/statistics 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 78/100
haskell/statistics#235 ·
-
难度 4/5 3-5 天 新手友好度 48/100
haskell/statistics#237 ·
-
难度 4/5 3-5 天 新手友好度 48/100
haskell/statistics#236 ·
-
难度 3/5 1-2 天 新手友好度 58/100
haskell/statistics#234 ·
-
难度 4/5 3-5 天 新手友好度 42/100
haskell/statistics#231 · 3 条评论 ·
查看 haskell/statistics 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复
-
Bug
难度 2/5 1-3 小时 新手友好度 68/100
-
pkg:crucible-mir type:bug
难度 2/5 1-3 小时 新手友好度 75/100
-
难度 2/5 1-3 小时 新手友好度 68/100
-
bug
难度 2/5 1-3 小时 新手友好度 78/100
GaloisInc/llvm-pretty#211 ·