diff --git a/.github/contributors/questoph.md b/.github/contributors/questoph.md new file mode 100644 index 000000000..24559c098 --- /dev/null +++ b/.github/contributors/questoph.md @@ -0,0 +1,106 @@ +# spaCy contributor agreement + +This spaCy Contributor Agreement (**"SCA"**) is based on the +[Oracle Contributor Agreement](http://www.oracle.com/technetwork/oca-405177.pdf). +The SCA applies to any contribution that you make to any product or project +managed by us (the **"project"**), and sets out the intellectual property rights +you grant to us in the contributed materials. The term **"us"** shall mean +[ExplosionAI GmbH](https://explosion.ai/legal). The term +**"you"** shall mean the person or entity identified below. + +If you agree to be bound by these terms, fill in the information requested +below and include the filled-in version with your first pull request, under the +folder [`.github/contributors/`](/.github/contributors/). The name of the file +should be your GitHub username, with the extension `.md`. For example, the user +example_user would create the file `.github/contributors/example_user.md`. + +Read this agreement carefully before signing. These terms and conditions +constitute a binding legal agreement. + +## Contributor Agreement + +1. The term "contribution" or "contributed materials" means any source code, +object code, patch, tool, sample, graphic, specification, manual, +documentation, or any other material posted or submitted by you to the project. + +2. With respect to any worldwide copyrights, or copyright applications and +registrations, in your contribution: + + * you hereby assign to us joint ownership, and to the extent that such + assignment is or becomes invalid, ineffective or unenforceable, you hereby + grant to us a perpetual, irrevocable, non-exclusive, worldwide, no-charge, + royalty-free, unrestricted license to exercise all rights under those + copyrights. This includes, at our option, the right to sublicense these same + rights to third parties through multiple levels of sublicensees or other + licensing arrangements; + + * you agree that each of us can do all things in relation to your + contribution as if each of us were the sole owners, and if one of us makes + a derivative work of your contribution, the one who makes the derivative + work (or has it made will be the sole owner of that derivative work; + + * you agree that you will not assert any moral rights in your contribution + against us, our licensees or transferees; + + * you agree that we may register a copyright in your contribution and + exercise all ownership rights associated with it; and + + * you agree that neither of us has any duty to consult with, obtain the + consent of, pay or render an accounting to the other for any use or + distribution of your contribution. + +3. With respect to any patents you own, or that you can license without payment +to any third party, you hereby grant to us a perpetual, irrevocable, +non-exclusive, worldwide, no-charge, royalty-free license to: + + * make, have made, use, sell, offer to sell, import, and otherwise transfer + your contribution in whole or in part, alone or in combination with or + included in any product, work or materials arising out of the project to + which your contribution was submitted, and + + * at our option, to sublicense these same rights to third parties through + multiple levels of sublicensees or other licensing arrangements. + +4. Except as set out above, you keep all right, title, and interest in your +contribution. The rights that you grant to us under these terms are effective +on the date you first submitted a contribution to us, even if your submission +took place before the date you sign these terms. + +5. You covenant, represent, warrant and agree that: + + * Each contribution that you submit is and shall be an original work of + authorship and you can legally grant the rights set out in this SCA; + + * to the best of your knowledge, each contribution will not violate any + third party's copyrights, trademarks, patents, or other intellectual + property rights; and + + * each contribution shall be in compliance with U.S. export control laws and + other applicable export and import laws. You agree to notify us if you + become aware of any circumstance which would make any of the foregoing + representations inaccurate in any respect. We may publicly disclose your + participation in the project, including the fact that you have signed the SCA. + +6. This SCA is governed by the laws of the State of California and applicable +U.S. Federal law. Any choice of law rules will not apply. + +7. Please place an “x” on one of the applicable statement below. Please do NOT +mark both statements: + + * [x] I am signing on behalf of myself as an individual and no other person + or entity, including my employer, has or will have rights with respect to my + contributions. + + * [ ] I am signing on behalf of my employer or a legal entity and I have the + actual authority to contractually bind that entity. + +## Contributor Details + +| Field | Entry | +|------------------------------- | -------------------- | +| Name | Christoph Purschke | +| Company name (if applicable) | University of Luxembourg | +| Title or role (if applicable) | | +| Date | 14/11/2019 | +| GitHub username | questoph | +| Website (optional) | https://purschke.info | diff --git a/spacy/cli/converters/conllu2json.py b/spacy/cli/converters/conllu2json.py index 8f2900a9b..e66a8c50e 100644 --- a/spacy/cli/converters/conllu2json.py +++ b/spacy/cli/converters/conllu2json.py @@ -34,6 +34,9 @@ def conllu2json(input_data, n_sents=10, use_morphology=False, lang=None, **_): doc = create_doc(sentences, i) docs.append(doc) sentences = [] + if sentences: + doc = create_doc(sentences, i) + docs.append(doc) return docs diff --git a/spacy/lang/it/punctuation.py b/spacy/lang/it/punctuation.py index 4439376c8..4fa931fde 100644 --- a/spacy/lang/it/punctuation.py +++ b/spacy/lang/it/punctuation.py @@ -5,7 +5,7 @@ from ..punctuation import TOKENIZER_INFIXES from ..char_classes import ALPHA -ELISION = " ' ’ ".strip().replace(" ", "").replace("\n", "") +ELISION = " ' ’ ".strip().replace(" ", "") _infixes = TOKENIZER_INFIXES + [ diff --git a/spacy/lang/lb/__init__.py b/spacy/lang/lb/__init__.py index 87058bdea..4fcfaddb4 100644 --- a/spacy/lang/lb/__init__.py +++ b/spacy/lang/lb/__init__.py @@ -3,6 +3,7 @@ from __future__ import unicode_literals from .tokenizer_exceptions import TOKENIZER_EXCEPTIONS from .norm_exceptions import NORM_EXCEPTIONS +from .punctuation import TOKENIZER_INFIXES from .lex_attrs import LEX_ATTRS from .tag_map import TAG_MAP from .stop_words import STOP_WORDS @@ -24,6 +25,7 @@ class LuxembourgishDefaults(Language.Defaults): tokenizer_exceptions = update_exc(BASE_EXCEPTIONS, TOKENIZER_EXCEPTIONS) stop_words = STOP_WORDS tag_map = TAG_MAP + infixes = TOKENIZER_INFIXES class Luxembourgish(Language): diff --git a/spacy/lang/lb/punctuation.py b/spacy/lang/lb/punctuation.py new file mode 100644 index 000000000..53df6bcd9 --- /dev/null +++ b/spacy/lang/lb/punctuation.py @@ -0,0 +1,16 @@ +# coding: utf8 +from __future__ import unicode_literals + +from ..punctuation import TOKENIZER_INFIXES +from ..char_classes import ALPHA + + +ELISION = " ' ’ ".strip().replace(" ", "") +HYPHENS = r"- – — ‐ ‑".strip().replace(" ", "") + + +_infixes = TOKENIZER_INFIXES + [ + r"(?<=[{a}][{el}])(?=[{a}])".format(a=ALPHA, el=ELISION) +] + +TOKENIZER_INFIXES = _infixes diff --git a/spacy/lang/lb/tokenizer_exceptions.py b/spacy/lang/lb/tokenizer_exceptions.py index adff9da36..18b58f2b1 100644 --- a/spacy/lang/lb/tokenizer_exceptions.py +++ b/spacy/lang/lb/tokenizer_exceptions.py @@ -2,31 +2,15 @@ from __future__ import unicode_literals from ...symbols import ORTH, LEMMA, NORM -from ..punctuation import TOKENIZER_PREFIXES # TODO -# tokenize cliticised definite article "d'" as token of its own: d'Kanner > [d'] [Kanner] # treat other apostrophes within words as part of the word: [op d'mannst], [fir d'éischt] (= exceptions) -# how to write the tokenisation exeption for the articles d' / D' ? This one is not working. -_prefixes = [ - prefix for prefix in TOKENIZER_PREFIXES if prefix not in ["d'", "D'", "d’", "D’"] -] - - _exc = { - "d'mannst": [ - {ORTH: "d'", LEMMA: "d'"}, - {ORTH: "mannst", LEMMA: "mann", NORM: "mann"}, - ], - "d'éischt": [ - {ORTH: "d'", LEMMA: "d'"}, - {ORTH: "éischt", LEMMA: "éischt", NORM: "éischt"}, - ], + } # translate / delete what is not necessary -# what does PRON_LEMMA mean? for exc_data in [ {ORTH: "wgl.", LEMMA: "wann ech gelift", NORM: "wann ech gelieft"}, {ORTH: "M.", LEMMA: "Monsieur", NORM: "Monsieur"}, @@ -64,6 +48,4 @@ for orth in [ ]: _exc[orth] = [{ORTH: orth}] - -TOKENIZER_PREFIXES = _prefixes TOKENIZER_EXCEPTIONS = _exc diff --git a/spacy/language.py b/spacy/language.py index 72044a0c5..266a1727d 100644 --- a/spacy/language.py +++ b/spacy/language.py @@ -678,7 +678,7 @@ class Language(object): kwargs = component_cfg.get(name, {}) kwargs.setdefault("batch_size", batch_size) if not hasattr(pipe, "pipe"): - docs = _pipe(pipe, docs, kwargs) + docs = _pipe(docs, pipe, kwargs) else: docs = pipe.pipe(docs, **kwargs) for doc, gold in zip(docs, golds): diff --git a/spacy/tests/lang/lb/test_text.py b/spacy/tests/lang/lb/test_text.py index 10cb16a41..387cf448d 100644 --- a/spacy/tests/lang/lb/test_text.py +++ b/spacy/tests/lang/lb/test_text.py @@ -5,18 +5,10 @@ import pytest def test_lb_tokenizer_handles_long_text(lb_tokenizer): - text = """Den Nordwand an d'Sonn - -An der Zäit hunn sech den Nordwand an d’Sonn gestridden, wie vun hinnen zwee wuel méi staark wier, wéi e Wanderer, deen an ee waarme Mantel agepak war, iwwert de Wee koum. Si goufen sech eens, dass deejéinege fir de Stäerkste gëlle sollt, deen de Wanderer forcéiere géif, säi Mantel auszedoen.", - -Den Nordwand huet mat aller Force geblosen, awer wat e méi geblosen huet, wat de Wanderer sech méi a säi Mantel agewéckelt huet. Um Enn huet den Nordwand säi Kampf opginn. - -Dunn huet d’Sonn d’Loft mat hire frëndleche Strale gewiermt, a schonn no kuerzer Zäit huet de Wanderer säi Mantel ausgedoen. - -Do huet den Nordwand missen zouginn, dass d’Sonn vun hinnen zwee de Stäerkste wier.""" + text = """Den Nordwand an d'Sonn An der Zäit hunn sech den Nordwand an d'Sonn gestridden, wie vun hinnen zwee wuel méi staark wier, wéi e Wanderer, deen an ee waarme Mantel agepak war, iwwert de Wee koum. Si goufen sech eens, dass deejéinege fir de Stäerkste gëlle sollt, deen de Wanderer forcéiere géif, säi Mantel auszedoen. Den Nordwand huet mat aller Force geblosen, awer wat e méi geblosen huet, wat de Wanderer sech méi a säi Mantel agewéckelt huet. Um Enn huet den Nordwand säi Kampf opginn. Dunn huet d'Sonn d'Loft mat hire frëndleche Strale gewiermt, a schonn no kuerzer Zäit huet de Wanderer säi Mantel ausgedoen. Do huet den Nordwand missen zouginn, dass d'Sonn vun hinnen zwee de Stäerkste wier.""" tokens = lb_tokenizer(text) - assert len(tokens) == 143 + assert len(tokens) == 142 @pytest.mark.parametrize( diff --git a/spacy/tests/regression/test_issue3001-3500.py b/spacy/tests/regression/test_issue3001-3500.py index b883ae67a..d05759c31 100644 --- a/spacy/tests/regression/test_issue3001-3500.py +++ b/spacy/tests/regression/test_issue3001-3500.py @@ -177,7 +177,6 @@ def test_issue3328(en_vocab): assert matched_texts == ["Hello", "how", "you", "doing"] -@pytest.mark.xfail def test_issue3331(en_vocab): """Test that duplicate patterns for different rules result in multiple matches, one per rule. @@ -328,6 +327,7 @@ def test_issue3449(): assert t3[5].text == "I" +@pytest.mark.filterwarnings("ignore::UserWarning") def test_issue3456(): # this crashed because of a padding error in layer.ops.unflatten in thinc nlp = English() diff --git a/spacy/tests/regression/test_issue3880.py b/spacy/tests/regression/test_issue3880.py index 6de373f11..c060473f5 100644 --- a/spacy/tests/regression/test_issue3880.py +++ b/spacy/tests/regression/test_issue3880.py @@ -2,8 +2,10 @@ from __future__ import unicode_literals from spacy.lang.en import English +import pytest +@pytest.mark.filterwarnings("ignore::UserWarning") def test_issue3880(): """Test that `nlp.pipe()` works when an empty string ends the batch. diff --git a/spacy/tests/regression/test_issue4348.py b/spacy/tests/regression/test_issue4348.py index 9391c3529..d2e27d563 100644 --- a/spacy/tests/regression/test_issue4348.py +++ b/spacy/tests/regression/test_issue4348.py @@ -3,8 +3,10 @@ from __future__ import unicode_literals from spacy.lang.en import English from spacy.util import minibatch, compounding +import pytest +@pytest.mark.filterwarnings("ignore::UserWarning") def test_issue4348(): """Test that training the tagger with empty data, doesn't throw errors""" diff --git a/spacy/tests/test_gold.py b/spacy/tests/test_gold.py index 731a1b5c2..fbdb3155b 100644 --- a/spacy/tests/test_gold.py +++ b/spacy/tests/test_gold.py @@ -177,8 +177,7 @@ def test_roundtrip_docs_to_json(): assert cats["BAKING"] == goldparse.cats["BAKING"] -# xfail while we have backwards-compatible alignment -@pytest.mark.xfail +@pytest.mark.skip(reason="skip while we have backwards-compatible alignment") @pytest.mark.parametrize( "tokens_a,tokens_b,expected", [ diff --git a/spacy/tests/test_language.py b/spacy/tests/test_language.py index d5398c145..7106cef74 100644 --- a/spacy/tests/test_language.py +++ b/spacy/tests/test_language.py @@ -65,6 +65,20 @@ def test_language_evaluate(nlp): nlp.evaluate([text, gold]) +def test_evaluate_no_pipe(nlp): + """Test that docs are processed correctly within Language.pipe if the + component doesn't expose a .pipe method.""" + + def pipe(doc): + return doc + + text = "hello world" + annots = {"cats": {"POSITIVE": 1.0, "NEGATIVE": 0.0}} + nlp = Language(Vocab()) + nlp.add_pipe(pipe) + nlp.evaluate([(text, annots)]) + + def vector_modification_pipe(doc): doc.vector += 1 return doc diff --git a/spacy/tests/tokenizer/test_urls.py b/spacy/tests/tokenizer/test_urls.py index 0e287aada..21e1819b7 100644 --- a/spacy/tests/tokenizer/test_urls.py +++ b/spacy/tests/tokenizer/test_urls.py @@ -55,10 +55,8 @@ URLS_SHOULD_MATCH = [ pytest.param( "chrome-extension://mhjfbmdgcfjbbpaeojofohoefgiehjai", marks=pytest.mark.xfail() ), - pytest.param("http://foo.com/blah_blah_(wikipedia)", marks=pytest.mark.xfail()), - pytest.param( - "http://foo.com/blah_blah_(wikipedia)_(again)", marks=pytest.mark.xfail() - ), + "http://foo.com/blah_blah_(wikipedia)", + "http://foo.com/blah_blah_(wikipedia)_(again)", pytest.param("http://⌘.ws", marks=pytest.mark.xfail()), pytest.param("http://⌘.ws/", marks=pytest.mark.xfail()), pytest.param("http://☺.damowmow.com/", marks=pytest.mark.xfail()), @@ -105,8 +103,8 @@ URLS_SHOULD_NOT_MATCH = [ "NASDAQ:GOOG", "http://-a.b.co", pytest.param("foo.com", marks=pytest.mark.xfail()), - pytest.param("http://1.1.1.1.1", marks=pytest.mark.xfail()), - pytest.param("http://www.foo.bar./", marks=pytest.mark.xfail()), + "http://1.1.1.1.1", + "http://www.foo.bar./", ] diff --git a/website/docs/usage/rule-based-matching.md b/website/docs/usage/rule-based-matching.md index 663ac5e5a..d3356f34c 100644 --- a/website/docs/usage/rule-based-matching.md +++ b/website/docs/usage/rule-based-matching.md @@ -986,37 +986,6 @@ doc = nlp("Apple is opening its first big office in San Francisco.") print([(ent.text, ent.label_) for ent in doc.ents]) ``` -### Adding IDs to patterns {#entityruler-ent-ids new="2.2.2"} - -The [`EntityRuler`](/api/entityruler) can also accept an `id` attribute for each -pattern. Using the `id` attribute allows multiple patterns to be associated with -the same entity. - -```python -### {executable="true"} -from spacy.lang.en import English -from spacy.pipeline import EntityRuler - -nlp = English() -ruler = EntityRuler(nlp) -patterns = [{"label": "ORG", "pattern": "Apple", "id": "apple"}, - {"label": "GPE", "pattern": [{"LOWER": "san"}, {"LOWER": "francisco"}], "id": "san-francisco"}, - {"label": "GPE", "pattern": [{"LOWER": "san"}, {"LOWER": "fran"}], "id": "san-francisco"}] -ruler.add_patterns(patterns) -nlp.add_pipe(ruler) - -doc1 = nlp("Apple is opening its first big office in San Francisco.") -print([(ent.text, ent.label_, ent.ent_id_) for ent in doc1.ents]) - -doc2 = nlp("Apple is opening its first big office in San Fran.") -print([(ent.text, ent.label_, ent.ent_id_) for ent in doc2.ents]) -``` - -If the `id` attribute is included in the [`EntityRuler`](/api/entityruler) -patterns, the `ent_id_` property of the matched entity is set to the `id` given -in the patterns. So in the example above it's easy to identify that "San -Francisco" and "San Fran" are both the same entity. - The entity ruler is designed to integrate with spaCy's existing statistical models and enhance the named entity recognizer. If it's added **before the `"ner"` component**, the entity recognizer will respect the existing entity @@ -1051,6 +1020,37 @@ The `EntityRuler` can validate patterns against a JSON schema with the option ruler = EntityRuler(nlp, validate=True) ``` +### Adding IDs to patterns {#entityruler-ent-ids new="2.2.2"} + +The [`EntityRuler`](/api/entityruler) can also accept an `id` attribute for each +pattern. Using the `id` attribute allows multiple patterns to be associated with +the same entity. + +```python +### {executable="true"} +from spacy.lang.en import English +from spacy.pipeline import EntityRuler + +nlp = English() +ruler = EntityRuler(nlp) +patterns = [{"label": "ORG", "pattern": "Apple", "id": "apple"}, + {"label": "GPE", "pattern": [{"LOWER": "san"}, {"LOWER": "francisco"}], "id": "san-francisco"}, + {"label": "GPE", "pattern": [{"LOWER": "san"}, {"LOWER": "fran"}], "id": "san-francisco"}] +ruler.add_patterns(patterns) +nlp.add_pipe(ruler) + +doc1 = nlp("Apple is opening its first big office in San Francisco.") +print([(ent.text, ent.label_, ent.ent_id_) for ent in doc1.ents]) + +doc2 = nlp("Apple is opening its first big office in San Fran.") +print([(ent.text, ent.label_, ent.ent_id_) for ent in doc2.ents]) +``` + +If the `id` attribute is included in the [`EntityRuler`](/api/entityruler) +patterns, the `ent_id_` property of the matched entity is set to the `id` given +in the patterns. So in the example above it's easy to identify that "San +Francisco" and "San Fran" are both the same entity. + ### Using pattern files {#entityruler-files} The [`to_disk`](/api/entityruler#to_disk) and