deepset-ai / FARM

:house_with_garden: Fast & easy transfer learning for NLP. Harvesting language models for the industry. Focus on Question Answering.
https://farm.deepset.ai
Apache License 2.0
1.74k stars 248 forks source link

Simplify tasks and connect_heads_with_processor usage/documentation #726

Closed Timoeller closed 2 years ago

Timoeller commented 3 years ago

The usage of tasks and how we need to connect PredictionHeads with Processors is non-obvious and needs to be simplified or better documented in example scripts for Multi Task learning (e.g. LM finetuning).

E.g. see error message in AdaptiveModel: "Label_tensor_names are missing inside the {head.task_name} Prediction Head. Did you connect the model" " with the processor through either 'model.connect_heads_with_processor(processor.tasks)'" " or by passing the processor to the Adaptive Model?"

In example lm_finetuning we neither call connect heads nor pass the processor to the adaptive model.

stale[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 21 days if no further activity occurs.

stale[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 21 days if no further activity occurs.