-
Couldn't load subscription status.
- Fork 15
Add NLTK Configuration to README.md (Fixes #25) #26
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
base: master
Are you sure you want to change the base?
Conversation
|
@ryzokuken I think there must be a way to download these packages through the command line itself.. rather than opening python REPL. Can you please use that instead! |
5aad9b6 to
5d1a5e7
Compare
|
Done. @526avijitgupta @yashmehrotra please review. |
|
@ryzokuken Please locally test your code before asking for code reviews |
|
I did test install and test and they were working perfectly |
|
@ryzokuken |
|
I realize that. Please help me figure out an alternative... It feels counterproductive to write a make command for just running the server as it requires just a single command anyway. Do I drop make serve and make run for now? |
|
@yashmehrotra I'm sorry I couldn't quite get what you meant to say (I did get upto reading their Makefile though). I was thinking, should I make the python server run in the background for |
|
Make a |
|
@yashmehrotra Could you please take on from here? The makefile stuff (saving PID in a temporary file on |
|
@yashmehrotra Are you going to work on this ? |
|
@ryzokuken @yashmehrotra Close this? |
|
@526avijitgupta why? I think I know more now, I can take care of the Makefiles. |
|
@ryzokuken That works too |
@yashmehrotra @526avijitgupta Please review.