Garry Bowlin

Results 9 comments of Garry Bowlin

Is there a reason why you are looking to add this particular widget? It looks like you don't have any exploration controls that use the widget, which is confusing. It...

related to #51, if we know the score type, this can be implicit rather than inferred.

In #55, I am adding a step to `make docs` that will clear out all notebooks contents. Its not a pre-commit hook, but it will run in the CI/CD pipeline.

For bounds I would assume we follow the numpy standard of include lower bound exclude upper bound, and if we need to we can come back with an option to...

Checked the updates to remove the in process notes around OTEL/Metrics. Made sure it runs with the new hieriarchial data from https://github.com/epic-open-source/seismometer-data/pull/8

@diehlbw this needs re-review because I've updated the metric recording to fix an issue with labels recorded to OTEL metrics.

> I can write it up as a separate issue, but re:OTEL "clean up": > > Do we really expect that 18 new ci jobs are needed around this? (all...

While #155 added a fair number of tests to pandas helpers, we still want to improve the quality of unit tests. In particular - we should try to come up...