Skip to content
Snippets Groups Projects
Commit 7a2f7973 authored by Maja Jablonska's avatar Maja Jablonska
Browse files

Remove unnecessary AllenNLP comments

parent 76a782fe
No related branches found
No related tags found
1 merge request!46Merge COMBO 3.0 into master
......@@ -64,17 +64,6 @@ class DatasetReader(IterableDataset):
yield instance
def _read(self) -> Iterable[Instance]:
"""
Reads the instances from the given `file_path` and returns them as an
`Iterable`.
You are strongly encouraged to use a generator so that users can
read a dataset in a lazy way, if they so choose.
"""
# NOTE: `file_path` is left untyped here on purpose.
# Technically the type should be `DatasetReaderInput`, but many subclass
# implementations of `DatasetReader` define their `_read()` method to take a more
# specific type, such as just `str`. But that would be a type error
# according to mypy: https://mypy.readthedocs.io/en/stable/common_issues.html#incompatible-overrides
raise NotImplementedError
def apply_token_indexers(self, instance: Instance) -> None:
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment