seismometer icon indicating copy to clipboard operation
seismometer copied to clipboard

minimal change for backwards compatibility post-otel

Open diehlbw opened this issue 5 months ago • 0 comments

Overview

A seismograph could break if it extends ConfigProvider, since otel setup expects an attribute that isnt initialized.

Description of changes

initialize config attr to None.
skip the otel initialization if None

Will write up issue (#176) for more integrated solution, likely a from_config method that handles the none case internally producing the NoOp manager.

Considered sufficient for a fix

Author Checklist

  • [ ] Linting passes; run early with pre-commit hook.
  • [ ] Tests added for new code and issue being fixed.
  • [ ] Added type annotations and full numpy-style docstrings for new methods.
  • [ ] Draft your news fragment in new changelog/ISSUE.TYPE.rst files; see changelog/README.md.

diehlbw avatar Aug 06 '25 14:08 diehlbw