Skip to content
This repository has been archived by the owner on Jan 13, 2022. It is now read-only.

FBNN package required? #14

Open
jeiranj opened this issue Sep 6, 2016 · 2 comments
Open

FBNN package required? #14

jeiranj opened this issue Sep 6, 2016 · 2 comments

Comments

@jeiranj
Copy link

jeiranj commented Sep 6, 2016

Hi,

First of all thanks for open sourcing this library and thanks for the clear readme. I appreciate it lots and lots.

I noticed that you require fbnn package insummary/nnlm.lua. I hear that it is not well maintained these days, and not only that, I do not see it used anywhere in the script. Can I just comment it out? Without it, it runs like a charm!

Thanks again and hope to hear from you soon.

@wangliangguo
Copy link

@jeiranj can you run it successfully?

@skrealworld
Copy link

Yes, you can.

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

No branches or pull requests

3 participants