-
Notifications
You must be signed in to change notification settings - Fork 17
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
jre has value 1.8 but 1.7 required and then CoreNLP needs 1.8+ #17
Comments
Thanks for the report. Can you explain what you mean by "1.7 required" and "edited registry to 1.7"? Also, which version of the JRE is actually installed? |
I have both jre 1.7 and 1.8 installed when PyStanfordDependencies threw the JavaVersionError "jre has value 1.8 that worked with respect to PyStanfordDependencies BUT led to an error in thanks BTW your tool promises to be very useful, all the Python wrappers on the On Wed, Sep 30, 2015 at 10:29 AM, David McClosky [email protected]
|
Can you send the full stacktrace for the "jre has value 1.8 but 1.7 required" error? PyStanfordDependencies on its own doesn't care about the JRE versions, it just tries to detect when the jars require a different version. Are you using subprocess or JPype for your backend? (maybe JPype needs JRE 1.7 on Windows?). |
OK - the program I run is stanford_dependencies_test.py (attached) If I have jre 1.8 set in the registry
http://stackoverflow.com/questions/26324486/properly-installing-java-8-along-with-java-7 I get dependencies_error.txt When I set the jre to 1.7 in the registry I get dependencies_error3.txt Thanks for looking at this. You can see I print out the command you execute On Wed, Sep 30, 2015 at 12:05 PM, David McClosky [email protected]
Command: java -ea -cp stanford-parser.jar edu.stanford.nlp.trees.EnglishGrammaticalStructure -basic -treeFile c:\users\crowner\appdata\local\temp\tmp3czf7c -keepPunct Command: java -ea -cp stanford-parser.jar edu.stanford.nlp.trees.EnglishGrammaticalStructure -basic -treeFile c:\users\crowner\appdata\local\temp\tmpsvzijl -keepPunct C:\Users\crowner\Desktop\doc\stanford-corenlp-full-2015-04-20>python stanford_de import os #os.environ['JAVAHOME'] = 'C:\Program Files\Java\jdk1.8.0_60\bin' import StanfordDependencies sent = sd.convert_tree('(S1 (NP (DT some) (JJ blue) (NN moose)))',debug=True) |
Hmm, sounds like having two different JREs on Windows is tricky. If it's at all possible, I would only have one version installed (though it sounds like StackOverflow users have found some workarounds). Yeah, PyStanfordDependencies doesn't look at |
Some good ideas, thanks. On Wed, Sep 30, 2015 at 2:26 PM, David McClosky [email protected]
|
One quick question that's probably obvious ;-) Why are you depending upon the Stanford jar? If you can take a parse tree Thanks On Wed, Sep 30, 2015 at 2:33 PM, Christopher Crowner [email protected]
|
It's a good question. PyStanfordDependencies doesn't actually know how to convert trees to dependencies, it just knows two different ways (subprocess and JPype) for calling the conversion code in Stanford CoreNLP/Stanford Parser. In other words, if there was a jar that just had the dependency conversion code in it and didn't include a parser, PyStanfordDependencies would be fine with that. It just uses Stanford CoreNLP/Parser jars by default since those are the ones that are typically distributed. |
Thanks, and thanks again for contributing something useful to the community! On Wed, Sep 30, 2015 at 2:44 PM, David McClosky [email protected]
|
edited registry to 1.7 then got
JavaRuntimeVersionError too old must use 1.8+ for CoreNLP
I am using the jar_filename parameter to point to the recent stanford-parser.jar
Thanks!
The text was updated successfully, but these errors were encountered: