-
Notifications
You must be signed in to change notification settings - Fork 325
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[v2] Merge MIEB and MTEB tasks #2078
Comments
Thanks for starting this! Updated to use question numberings. Tagging @gowitheflow-1998 while I take a deeper look into the AbsTasks. A few points to begin with:
|
|
|
|
thanks for starting this!
I personally think keeping them as is (instead of combining with Any2AnyRetrieval) will be less confusing to people (if they try to add more MC tasks)? |
All logic with qrels is hidden in evaluator. We can make evaluator as parameter of class and with this all logic in |
v2
, we mergedRetrieval
andReranking
tasks, andMultiChoice
appears to be aReranking
task with a single correct answer, can we merge it with theRetrieval
task?MultiChoiceAny2Any
, do we still needMultiChoiceAny2Text
?AbsTaskClassification
and split it intoAbsTaskTextClassification
andAbsTaskImageClassification
, since they share the same logic in theevaluate
function?ImageTextSimilarity
—should we combine them?Any2AnyPairClassification
andAny2AnySTS
.Ref:
The text was updated successfully, but these errors were encountered: