Skip to content
This repository was archived by the owner on Apr 8, 2025. It is now read-only.

Ensure QAInferencer always has task_type "question_answering"#460

Merged
brandenchan merged 3 commits intomasterfrom
qa_inf_task
Jul 15, 2020
Merged

Ensure QAInferencer always has task_type "question_answering"#460
brandenchan merged 3 commits intomasterfrom
qa_inf_task

Conversation

@brandenchan
Copy link
Copy Markdown
Contributor

Though the parent Inferencer class can set task_type to one of many values, it should always be "question_answering" in the QAInferencer. This PR enforces this and gives a warning when a user tries to initialise the class with a different value

@brandenchan brandenchan requested a review from Timoeller July 14, 2020 14:32
Copy link
Copy Markdown
Contributor

@Timoeller Timoeller left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looking good. Minor change in the warning message needed.

Lets change this once we have multiple types of QA styles.

@brandenchan brandenchan merged commit 1616f29 into master Jul 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants