adapter-bert icon indicating copy to clipboard operation
adapter-bert copied to clipboard

missing processors

Open jacobdeasy opened this issue 4 years ago • 1 comments

Congratulations on the great paper!

One question, do you have additional processor classes? At the moment, the code reads:

`processors = {
      "cola": ColaProcessor,
      "mnli": MnliProcessor,
      "mrpc": MrpcProcessor,
  }`

and later:

`if task_name not in processors:
    raise ValueError("Task not found: %s" % (task_name))`

meaning that only 3 datasets can be used for training.

It would be useful for replication purposes to have all processors available. Let me know if this is possible or if I have misunderstood!

jacobdeasy avatar Jul 03 '20 10:07 jacobdeasy

run_classifier.py just serves as an example, and should be relatively easy to extend to other tasks. See, for example, the original BERT codebase (https://github.com/google-research/bert/) for an XNLI preprocessor, or for a separate main for SQuAD Question Answering.

neilhoulsby avatar Jul 05 '20 13:07 neilhoulsby