The `TrainerModel` class should contain only one default model

vindolineKis / QCircLearning

MIT License

0 stars 0 forks source link

The `TrainerModel` class should contain only one default model #2

Open Canoming opened 2 days ago

Canoming commented 2 days ago

Including multiple default models as staticmethod is confusing. We can provide these models as a subclass of TrainerModel and init it with super().

https://github.com/vindolineKis/QCircLearning/blob/25c1983882feefe685bf0a27dec32eebd6593b10/QCircLearning/trainerModel.py#L45