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

关于另外一个predict和train的bug (seq2seq pytorch) #146

Open
JGanson opened this issue Sep 23, 2022 · 9 comments
Open

关于另外一个predict和train的bug (seq2seq pytorch) #146

JGanson opened this issue Sep 23, 2022 · 9 comments

Comments

@JGanson
Copy link

JGanson commented Sep 23, 2022

我看target_Lang 是记录所有ask的input,但在predict里:
input_tensor = tensorFromSentence(input_lang,sentence)

我觉得这里第一个参数应该是target_lang, 因为client是向robot问话

train 也是同样的问题,把target和input搞反了

@JGanson JGanson changed the title 关于另外一个predict的bug (seq2seq pytorch) 关于另外一个predict和train的bug (seq2seq pytorch) Sep 23, 2022
@zzzerow
Copy link

zzzerow commented Oct 16, 2022

对,这个很迷惑

@JGanson
Copy link
Author

JGanson commented Oct 16, 2022

对,这个很迷惑
估计他写完可能忘记debug了。我自己把他的code整理了一下,改成了一个英文版的chatbot。修好bug之后跑起来是没有问题的

@zzzerow
Copy link

zzzerow commented Oct 17, 2022

@JGanson
Copy link
Author

JGanson commented Oct 17, 2022

这个代码还没有并行处理所以训练比较慢,可以参考这里
对的,我也是好奇这个。他在train里面是一个for loop去遍历训练。但我不知道怎么去修改

@JGanson
Copy link
Author

JGanson commented Oct 17, 2022

我看了一下你的这个链接,感觉可以改一下。

@JGanson
Copy link
Author

JGanson commented Oct 18, 2022

这个代码还没有并行处理所以训练比较慢,可以参考这里
我研究了一下,感觉这个得需要大改这整个model。有点超出我的范围了 :<

@zhaoyingjun
Copy link
Owner

感谢两位指出,我回头找时间review一下代码,包括并行处理 都解决一下,预计10月30日更新。

@JGanson
Copy link
Author

JGanson commented Oct 19, 2022

感谢两位指出,我回头找时间review一下代码,包括并行处理 都解决一下,预计10月30日更新。

没事,非常感谢你的project。我自己做了一个英文版的chatbot是基于你给的model改的。非常感谢你的project :》

@Zx1999
Copy link

Zx1999 commented Nov 7, 2022

感谢两位指出,我回头找时间review一下代码,包括并行处理 都解决一下,预计10月30日更新。
请问近期有更新的打算嘛 :)

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

No branches or pull requests

4 participants