Depricating ModelAdapter
I do not think that ModelAdapter is useful anymore.
- https://ontolearn-docs-dice-group.netlify.app/autoapi/ontolearn/model_adapter/#ontolearn.model_adapter.ModelAdapter
- https://ontolearn-docs-dice-group.netlify.app/autoapi/ontolearn/model_adapter/index.html#ontolearn.model_adapter.ModelAdapter
- https://ontolearn-docs-dice-group.netlify.app/usage/03_examples.html#ex-4-using-model-adaptor
@sapkotaruz11 @alkidbaci shall we remove it in our next release ?
Model adapter is supposed to ease the initialization of a model but as you do, I also think there is barely anyone finding this useful enough. I guess its okay to remove it.
Should not remove the whole model_adapter.py module though because there are a few other useful methods/classes there. Maybe we can rename the module to something else or move the useful code to some other module.
Ideally, we should remove the script. As those useful methods/classes defined there are not related to model_adapter, e.g. Trainer. Yet, for the time being, we can only focus on deleting the ModelAdapter class along with the related tests
Should I take this on?
please do
Maybe it would be nice to rename the script as well. The execute function in the model_adapter is directly called from the main.py, and once we remove the model_adapter function, the naming of the script might seem somewhat confusing.