seismometer icon indicating copy to clipboard operation
seismometer copied to clipboard

Create more standard path for extending config

Open diehlbw opened this issue 4 months ago • 0 comments

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

diehlbw avatar Sep 12 '25 14:09 diehlbw