capitalone / rubicon-ml

Capture all information throughout your model's development in a reproducible way and tie results directly to the model code!
https://capitalone.github.io/rubicon-ml/
Apache License 2.0
127 stars 32 forks source link

fix default `MetricListsComparison.column_names` #470

Closed ryanSoley closed 1 month ago

ryanSoley commented 1 month ago

Describe the bug

when MetricListsComparison.column_names is the default None, there is a check against its len that raises an error

Steps/Code to reproduce bug

mlc = MetricListsComparison(experiments, column_names=None)
mlc.serve()  # raises error

Expected behavior

there should just be no column names on the viz rather than raising an error

we just need to set column_names to an empty list in the init if it is passed in as None