[reinhold-willcox]

The get_event_strings function (and the related subfunctions) now allow for an optional seeds argument to be passed. Previously, these functions would only operate on the entire COMPAS dataset, which can become quite slow for large datasets.

There is also some minor tidying to the module.

[ilyamandel]

Thanks, @reinhold-willcox . I am not a regular python user, but happy to approve since nobody else is jumping in. :)

[reinhold-willcox]

Thanks Ilya! There's something going wrong in the pychecks though, I'll fix that first.

[ilyamandel]

Hi @reinhold-willcox , @avivajpeyi

Any luck in fixing the pytests? I am happy for you to pull this in, regardless.

---

=========================== short test summary info ============================
FAILED py_tests/test_debugging_utils.py::test_get_event_history_and_event_strings - TypeError: object of type 'numpy.int64' has no len()
= 1 failed, 50 passed, 1 skipped, 1 deselected, 7 warnings in 62.21s (0:01:02) =
Error: Process completed with exit code 1.