Make JsonlReader path optional (#8396)

To avoid config errors during training when `[corpora.pretrain.path]` is
`None` with the default `spacy.JsonlCorpus.v1` reader, make the reader
path optional, similar to `spacy.Corpus.v1`.
This commit is contained in:
Adriane Boyd 2021-06-15 14:55:15 +02:00 committed by GitHub
parent 94e1346f44
commit 480a3bf3be
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 1 deletions

View File

@ -41,7 +41,7 @@ def create_docbin_reader(
@util.registry.readers("spacy.JsonlCorpus.v1") @util.registry.readers("spacy.JsonlCorpus.v1")
def create_jsonl_reader( def create_jsonl_reader(
path: Path, min_length: int = 0, max_length: int = 0, limit: int = 0 path: Optional[Path], min_length: int = 0, max_length: int = 0, limit: int = 0
) -> Callable[["Language"], Iterable[Doc]]: ) -> Callable[["Language"], Iterable[Doc]]:
return JsonlCorpus(path, min_length=min_length, max_length=max_length, limit=limit) return JsonlCorpus(path, min_length=min_length, max_length=max_length, limit=limit)