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

Tika 1913 - MIT Information Extraction itegrated with Tika #108

Merged
merged 7 commits into from Apr 23, 2016

Conversation

manalishah
Copy link
Contributor

@manalishah manalishah commented Apr 22, 2016

This pull request comprises of yet another NamedEntityRecognizer that uses the open-source trained models and functions of MIT-nlp to perform information extraction. I have provided it as a runtime binding to MITIE.

In order to test it out, you can download mitie-resources and follow the steps or build it from source from mit-nlp/MITIE

@chrismattmann


import java.io.File;
import java.lang.reflect.Method;
import java.util.*;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

replace * imports with actual imports

@chrismattmann
Copy link
Contributor

@manalishah

@manalishah
Copy link
Contributor Author

@chrismattmann done :)

@chrismattmann
Copy link
Contributor

thanks @manalishah will merge soon.

LOG.warn("{} does not exist", modelPath);
}else {
Class<?> namedEntityExtractorClass = Class.forName(NamedEntityExtractor_Class);
extractorInstance = namedEntityExtractorClass.getDeclaredConstructor(new Class[]{String.class}).newInstance("/Users/manali/cs599_dr/MITIE/MITIE-models/english/ner_model.dat");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@manalishah
Please use path from function parameter instead of the hardcoded string.
Perhaps you forgot to modify in last moment ;-) ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh no!!! @thammegowda thanks! its fixed now x_x

@asfgit asfgit merged commit cd06762 into apache:master Apr 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants