georgemilosh / Climate-Learning

How to predict extreme events in climate using rare event algorithms and modern tools of machine learning
https://georgemilosh.github.io/Climate-Learning/
MIT License
20 stars 5 forks source link

Support for cascading inheritance #82

Closed AlessandroLovo closed 9 months ago

AlessandroLovo commented 10 months ago

RIght now it is easy to inherit from Learn2_new thanks to the inheritance_template. However, it can be messy to inherit from one or multiple children of Learn2_new.

For example, intrinsically_interpretable_probabilistic_regression could benefit from inheriting from probabilistic_regression and committor_projection_NN. Instead, right now, it inherits directly from Learn2_new, with the consequence that much of its code is duplicated with its similar siblings. This has already made maintaining the code troublesome.

It would then be interesting to implement more flexibility for the inheritance

AlessandroLovo commented 9 months ago

After some tests, everything seems to be working fine