Skip to content
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

Enables the use of versions higher than 3.4.1 #55

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Enables the use of versions higher than 3.4.1 #55

wants to merge 2 commits into from

Conversation

DanielJamieson
Copy link

Adds a parameter to the Stanford CoreNLP constructor called "version_str", which takes on the default value of "3.4.1".

This variable, when combined with with the "corenlp_path" variable, allows the usage of more recent versions of the coreNLP library without major refactoring of the code base. Simple pass the path of the library as well as the version string so that this python wrapper can successfully find and load the correct JAR files.

This change does not guarantee full compatibility with all versions and future releases of the Stanford CoreNLP Library - but has been used with up to the current latest version at this time (3.8.0) with no problems.

…ng the version of the library. Enables the use of versions higher than 3.4.1
… be ran and imported from files outside of the root folder. Added command line arguments for corenlp path & version string
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant