graykode / graykode/nlp-tutorial

3-3.Bi-LSTM may have wrong padding

未关闭
#72 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
Jupyter Notebook
星标
14.9k
派生
3.9k
PR 合并指标
30 天内没有已合并 PR

描述

In line 16 you use
` input = input + [0] * (max_len - len(input))`
the padding, you use 0, which means the first word 'Lorem'.
but it is not the right choose.
I think you can change like that
```
# word_dict = {w: i for i, w in enumerate(list(set(sentence.split())))}
# number_dict = {i: w for i, w in enumerate(list(set(sentence.split())))}
word_dict = {w: i for i, w in enumerate(['PAD']+list(set(sentence.split())))}
number_dict = {i: w for i, w in enumerate(['PAD']+list(set(sentence.split())))}
```

贡献指南

打开贡献指南

调研方向

Open the 3-3.Bi-LSTM entry and inspect line 16, where padding is appended to inputs. Verify that the padding index is distinct from the first word and is used consistently; the issue is done when padded inputs no longer map to “Lorem” and the example still runs.

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

评估

技术栈
jupyter-notebook
领域
machine-learning
Issue 类型
缺陷
难度
2/5
预计耗时
1-3 小时
活跃度
停滞
描述清晰度
基本清楚
新手友好度
35/100

把新 issue 发到你的邮箱

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