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

Modernize, i.e., update to processors using Stanford CoreNLP 4.4.0 #772

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

Conversation

kwalcock
Copy link
Member

This implies an update to Lucene 7.5.0. This is a draft used to explore the repercussions.

@kwalcock
Copy link
Member Author

Failing tests include TestFastBioNLPProcessor in processors/test

[info] - should parse dependencies correctly *** FAILED *** (18 milliseconds)
[info]   false was not true (TestFastBioNLPProcessor.scala:147)

@kwalcock
Copy link
Member Author

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.

[info] *** 559 TESTS FAILED ***
[error] Failed tests:
[error] 	org.clulab.reach.DemoTests2
[error] 	org.clulab.reach.TestMentionSerialization
[error] 	org.clulab.reach.TestBindingEvents
[error] 	org.clulab.reach.TestNERLabeling
[error] 	org.clulab.reach.TestGrounding
[error] 	org.clulab.reach.DemoTests3
[error] 	org.clulab.reach.PolaritySuite
[error] 	org.clulab.reach.TestDeModifications
[error] 	org.clulab.reach.TestTranscriptionEvents
[error] 	org.clulab.reach.TestTemplaticSimpleDeEvents
[error] 	org.clulab.reach.TestTemplaticAutoEvents
[error] 	org.clulab.reach.TestPolarity
[error] 	org.clulab.reach.TestHyphenedEvents
[error] 	org.clulab.reach.NegationTests
[error] 	org.clulab.reach.TestActivationEvents
[error] 	org.clulab.reach.TestCoreference
[error] 	org.clulab.reach.TestTranslocationEvents
[error] 	org.clulab.reach.TestReachGrounder
[error] 	org.clulab.reach.DemoTests1
[error] 	org.clulab.reach.TestModifications
[error] 	org.clulab.reach.ExperimentalRegulationTests
[error] 	org.clulab.reach.TestRegulationEvents
[error] 	org.clulab.reach.TestTemplaticSimpleEvents
[error] 	org.clulab.reach.TestDependencyUtils
[error] 	org.clulab.reach.TestConversionEvents

@kwalcock
Copy link
Member Author

For causalAssembly/test:

[info] *** 21 TESTS FAILED ***
[error] Failed tests:
[error] 	org.clulab.reach.assembly.TestAssemblySieves
[error] 	org.clulab.reach.assembly.TestAssemblyManager

@kwalcock
Copy link
Member Author

For export/test

[info] *** 27 TESTS FAILED ***
[error] Failed tests:
[error] 	org.clulab.reach.export.TestFeedback2
[error] 	org.clulab.reach.export.TestApi
[error] 	org.clulab.reach.export.TestFriesOutput
[error] 	org.clulab.reach.export.TestOutputDegrader

@enoriega
Copy link
Member

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 Document class won't change. But we should make sure we don't loose the ability of de-serializing files

@kwalcock
Copy link
Member Author

@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.

@enoriega
Copy link
Member

Ah! I saw a notification on an email and ended up here by mistake. Sorry!

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.

None yet

2 participants