-
Notifications
You must be signed in to change notification settings - Fork 39
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
Modernize, i.e., update to processors using Stanford CoreNLP 4.4.0 #772
base: master
Are you sure you want to change the base?
Conversation
Failing tests include TestFastBioNLPProcessor in processors/test
|
It's much worse for main/test. Note that I did not recalculate any indexes or whatever they are. I have to look up what that was about.
|
For causalAssembly/test:
|
For export/test
|
Hi @kwalcock . Will this change be backwards compatible with serialized files that have already been annotated with an older version of processors? (i.e. the serialized papers from PubMed). I think it will, because the |
@enoriega, was this comment for a different PR? This one is a year old. I should probably track down a unit test that checks on the serialization to be sure. |
Ah! I saw a notification on an email and ended up here by mistake. Sorry! |
This implies an update to Lucene 7.5.0. This is a draft used to explore the repercussions.