github / github/rust-gems

Allow empty `ConsistentPermutation`

未关闭 适合新手
#159 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
Rust
星标
134
派生
24
平均合并
16 小时 27 分钟
30 天内合并 PR
11

描述

Is there any reason `ConsistentPermutation` doesn't allow `n` to be zero - as per [this assertion](https://github.com/github/rust-gems/blob/main/crates/consistent-choose-k/src/consistent_permutation.rs#L156).

This results in special casing when you're constructing the permutations. We might need a branch at the top of the `next` function to check, but it should be highly predictable.

贡献指南

打开贡献指南

调研方向

从 crates/consistent-choose-k/src/consistent_permutation.rs 开始,重点查看第 156 行的 assertion 以及 issue 中提到的下一个函数。验证当前如何处理 n = 0,并确认无需在调用方进行特殊处理即可构造空排列。

由索引模型根据 Issue 内容生成。

评估

技术栈
rust
领域
tooling
Issue 类型
功能
难度
2/5
预计耗时
1-3 小时
活跃度
活跃
描述清晰度
描述清楚
新手友好度
82/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。