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

Jump to bottom

Fix JMLH bugs #2

Merged

ymcidence merged 1 commit into ymcidence:master from TreezzZ:master

Apr 3, 2020

TreezzZ commented Apr 2, 2020

您好！感谢您开源的代码，我在运行JMLH的代码时候发现按照您的步骤无法运行，排查后发现

./model/jmlh.py的20行和28行，由于binary_activation.binary_activation返回的是一个tuple，造成后续输入FC层出现错误。
./train/jmlh_train.py的29行，batch_data的第2项为标签数据，您这里写成3了，造成了数组下标错误。

再次感谢您的工作！


          Fix JMLH bugs

2cccdc2

ymcidence merged commit 778dd1c into ymcidence:master

Owner

ymcidence commented Apr 3, 2020

您好！感谢您开源的代码，我在运行JMLH的代码时候发现按照您的步骤无法运行，排查后发现

./model/jmlh.py的20行和28行，由于binary_activation.binary_activation返回的是一个tuple，造成后续输入FC层出现错误。

./train/jmlh_train.py的29行，batch_data的第2项为标签数据，您这里写成3了，造成了数组下标错误。

再次感谢您的工作！

感谢TreezzZ对于bug的修复，JMLH的脚本我在完成后没有经过太多测试，造成使用上的困难，为此深表歉意

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment