Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix for #759 #772

Merged
merged 2 commits into from
May 20, 2021
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion farm/data_handler/processor.py
Original file line number Diff line number Diff line change
Expand Up @@ -570,6 +570,7 @@ def __init__(
self.skiprows = skiprows
self.header = header
self.max_samples = max_samples
logger.warning(f"Currently no support in Processor for returning problematic ids")

super(TextClassificationProcessor, self).__init__(
tokenizer=tokenizer,
Expand Down Expand Up @@ -672,7 +673,6 @@ def dataset_from_dicts(self, dicts, indices=None, return_baskets=False, debug=Fa

# TODO populate problematic ids
problematic_ids = set()
logger.warning("Currently no support in Processor for returning problematic ids")
dataset, tensornames = self._create_dataset()
if return_baskets:
return dataset, tensornames, problematic_ids, self.baskets
Expand Down