NLPbox / stanford-corenlp-docker

build/run the most current Stanford CoreNLP server in a docker container
43 stars 32 forks source link

Fix Stanford CoreNLP download link #7

Closed RafaelPiloto10 closed 5 months ago

RafaelPiloto10 commented 5 months ago

It seems like the previous download link no longer exists or is broken -- this PR patches it to a forced URL at v4.5.6. I was unable to find a general "latest" url, but this will at the least get the docker version back up and running until a longer term solution is supported.

arne-cl commented 5 months ago

Dear Rafael,

thanks for the heads up! I would like to avoid pinning the CoreNLP version in the Dockerfile, so I'll try to come up with a different solution. How would I use the docker-publish.yml action?

RafaelPiloto10 commented 5 months ago

Thank you for your response -- I see a fix has been posted -- thank you! Please ignore the docker-publish.yml action which simply builds and serves the image. I believe you are currently serving the image through the docker registry, which is fine. The action would simply automate the image building process and serve it through GitHub's container registry instead, which is what we were doing temporarily.

With my issue now resolved, I will close this PR. Thank you for your help!