search

neulab / ExplainaBoard

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

Remove serializable from processors #522

Closed odashi closed 1 year ago

odashi commented 1 year ago

This is a follow-up change of #422.

Processors are not serialized in any data. This change removes the Serializable interface from all processors to avoid unnecessary confusion in the common registry.

odashi commented 1 year ago

@tetsuok I noticed that there are several bugs around the registry in #521, and similar bugs may be involved in the future in the Processor's factory too. I decided to implement unit tests around Processors as well as you first implemented in #434 regardless of its cost. Please take a look at this change to check if they are implemented appropriately.