alteryx / alteryx/nlp_primitives

Potentially unneeded memory alloc in `StopwordCount`

未关闭
#186 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
Python
星标
37
派生
13
PR 合并指标
30 天内没有已合并 PR

描述

I don't think we need to create a list in this comprehension:
https://github.com/alteryx/nlp_primitives/blob/11837a50de79fd05e067f58075100f39f639e563/nlp_primitives/stopword_count.py#L42

Instead, I think we can just iterate over `words` and keep a `count` variable. For very large strings this would save the time and memory needed to alloc the list.

贡献指南

这个仓库没有索引到贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

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