Skip to content

Issue #4: ExactMatchMultiClassDictionaryNER implementation and test #5

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 64 commits into
base: master
Choose a base branch
from
Open

Issue #4: ExactMatchMultiClassDictionaryNER implementation and test #5

wants to merge 64 commits into from

Conversation

sujitpal
Copy link

This patch addresses #4

@sujitpal sujitpal mentioned this pull request Oct 28, 2019
Open
@sujitpal sujitpal mentioned this pull request Oct 29, 2019
Open
sujitpal added 23 commits October 29, 2019 08:08
@sujitpal
NERDS-4: added a fit() method to ExactMatchMultiClassDictionaryNER
98f8797
@sujitpal
moving the current nerds package to nerds_orig
ce26f2f
@sujitpal
adding ExactMatchMultiClassDictionaryNER entry to __init__.py
aa7b257
@sujitpal
streamlined nerds models and code with anago style data backbone
5f43d9d
@sujitpal
Initial commit for Ensemble NER (maximal hard voting) with weights pl…
fe0de32 
…us test.
@sujitpal
replacing mention of IOB with BIO to reduce confusion.
08c3ef3
@sujitpal
Replaced deprecated references to sklearn.externals.joblib with jobli…
0d352f8 
…b per deprecation warning, added load/save tests to CRF and Dictionary NER models.
@sujitpal
Adding load and save for all NER models to tests.
f12f915
@sujitpal
flatten and unflatten lists with tests
95bd584
@sujitpal
GMB example rewrite with updated API
956c13a
@sujitpal
fix section levels
692c6b9
@sujitpal
also remove zip file
3025e85
@sujitpal
moving artifacts to dedicated docs directory
1885d92
@sujitpal
remove version capping for SpaCy
67171ae
@sujitpal
remove glove embeddings file during cleanup
172ce09
@sujitpal
updating README to describe forked code
39d2c96
@sujitpal
example usage on BioNLP dataset
48cdb3f
@sujitpal
uncommenting commented code
156bf89
@sujitpal
API update (transform -> predict)
7cc402e
@sujitpal
remove NLTK, add allenNLP (for ElmoNER) and transformers
538069a
@sujitpal
spelling fix
1a9dccc
@sujitpal
added utility functions to convert back and forth from tokens-tags to…
e36981f 
… sentence-spans representations.
@sujitpal
cosmetic change: added line before def
fdc13c1
@sujitpal
replace vocab building using LabelEncoder
d3cf616
@sujitpal
Ensemble NER can run fit() and predict() in parallel
26d62a3
@sujitpal
update to n_jobs usage in Ensemble NER
24b3b56
@sujitpal
documentation changed to numpy docstring format.
b6e9e2a
@sujitpal
updated list of changes
9652959
@sujitpal
documentation fix
008ffb9
@sujitpal
integrated FLAIR SequenceTagger (NER) into NERDS.
3539808
@sujitpal
added FLAIR to both examples, updated Ensemble calling (constructor i…
78ae769 
…njection) style.
@sujitpal
documentation fixes
744622e
@sujitpal
documentation fixes
396fd6f
@sujitpal
params.yaml file creation in save().
e0f47c0
@sujitpal
Flair NER additions
c3ba6fb
@sujitpal
added link to blog post on flair based NER
aa96dd4
@sujitpal
updated link to BioNLP shared task
c2751f8
@sujitpal
documentation change
49b8511
@sujitpal
updated link to data
b7bfdc3
@sujitpal
rename tests to reflect intent (fix copy-paste)
9f21095
@sujitpal
add encoding to load_data_and_labels
dff1a70
@sujitpal
initial revision
5d90712
@sujitpal
fix default behavior to pad with padding tag in case of truncated tok…
c47d8ef 
…ens during prediction, with utility method to align for metric computations
@sujitpal
added suppress warnings to all tests, added new utils methods and tests.
2a6cc00
@sujitpal
initial revision for simpletransformer based NER
a94f169
@sujitpal
adding example runs
22caa4c
@sujitpal
uncommenting clean up
35ef8de
@sujitpal
adding classification report for TransformerNER
c9f5578
@sujitpal
updated change list to include TransformerNER
58c556c
@sujitpal
adding TransformerNER blog post link
607bde5
@sujitpal
updated preprocessing logic and marked it experimental
514ac22
@sujitpal
predict does its own batching, more efficient without external batching
951fdc4
@sujitpal
Updated Pipfile based on security vulnerability, removing Pipfile.loc…
366420b 
…k since no longer using.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@sujitpal