DynaDojo / dynadojo

An Extensible Benchmarking Platform for Scalable Dynamical System Identification
https://dynadojo.github.io/dynadojo/
MIT License
14 stars 11 forks source link

improve: use decorators rather than wrapper classes #44

Open carynbear opened 3 months ago

carynbear commented 3 months ago

The issue with AlgorithmChecker and SystemChecker is that methods/attributes are not forwarded to the wrapped class which reduces the flexibility of DynaDojo.

For example, you cannot access an underlying Tensorflow or Pytorch neural net model without first calling ._alg which means you can't move the model to GPU natively.

I think we should migrate to using decorators.

Relevant stackoverflow post: https://stackoverflow.com/a/63644860