epic-open-source / seismometer

AI model evaluation with a focus on healthcare

https://epic-open-source.github.io/seismometer/

BSD 3-Clause "New" or "Revised" License

179 stars 21 forks source link

Update Plotting Controls to allow Exploration of Settings #41

Closed gbowlin closed 2 months ago

gbowlin commented 3 months ago

Overview

This is a Draft to get some developer feedback on some larger changes around exploration of model data.

This is a pretty big change to allow model exploration by wrapping plots in user friendly controls rather than setting values directly on the Seismogram Singleton.

Description of changes

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.

andli28 commented 2 months ago

test_thesholds.py should be spelled test_thresholds.py