Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename word2vec_helpers.py to word2vec_helpers9.28.py #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Rename word2vec_helpers.py to word2vec_helpers9.28.py #9

wants to merge 1 commit into from

Conversation

chenzk1993
Copy link

作者你好,你这个函数有些问题,原程序中sentence处理后并没有赋给新的padding_sentences,而你返回时,返回的却是sentences,所以相当于上面操作是没用的。

作者你好,你这个函数有些问题,原程序中sentence处理后并没有赋给新的padding_sentences,而你返回时,返回的却是sentences,所以相当于上面操作是没用的。
@clayandgithub
Copy link
Owner

首先在此表示感谢!
但经过我仔细检查和测试,原程序中的padding_sentences其实没有这样的问题。
在for sentence in sentences这个循环中,对于每个sentence的更改,实际就是对sentences的更改,因此返回sentences是没有问题的。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants