seismometer
seismometer copied to clipboard
Create more standard path for extending config
Issue
There are many cases where it's desirable to add extra configuration. Extending base classes and overriding methods only gets so far, and is susceptible to break across versions if there are changes made to the base function. Calling super likely creates need for redundancy.info
Describe the solution you'd like
OtherInfo seems a reasonable grouping for such model-specific extensions so, instead of hardcoding the OtherInfo class allow for DIP by having a class attribute that then gets instantiated
i.e. Make this line refer to an injectable/overridable parameter: https://github.com/epic-open-source/seismometer/blob/main/src/seismometer/configuration/config.py#L124