-
-
Notifications
You must be signed in to change notification settings - Fork 89
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
Add Docker support for the server #67
Comments
thanks for your mail and the issue report - i just read this issue after getting your mail. could you let me know, what issues were you facing that made starting and running the corenlp server "impractical"? we do provide a simple command (see readme.md) to startup the server so im unsure how this is not practical. or did you face an issue when starting up the server? |
I had a few issues:
|
I understand your comments. In general, I want to keep our repos as lean as possible, i.e., so that they focus on only the core functionality (which here is the 5W1H extraction). Only for convenience, we added this "download and run" command for the server. Having two ways of doing this function also means having two things to support and generally I try to avoid redundancy. However, in case the docker version is more convenient, I might add it and perhaps remove the current way Feel free to open a PR for me and my colleagues to have a look :) |
run the NLP server inside a docker container to make running the server practical
The text was updated successfully, but these errors were encountered: