neulab / ExplainaBoard

Interpretable Evaluation for AI Systems
MIT License
361 stars 36 forks source link

Implement Loader factory #521

Closed odashi closed 2 years ago

odashi commented 2 years ago

Fixes #421

This pr implements a factory to obtain Loader classes, and removes the loader registry.

odashi commented 2 years ago

@neubig This is the same situation that I/ @tetsuok / @pfliu-nlp discussed previously:

https://github.com/neulab/ExplainaBoard/pull/434#issuecomment-1255500133

At last, we eventually have no reason to adopt the common registry for Loader functionalities at this point.