I would appreciate if you kept the module self-contained, which would mean keeping the if __name__ == '__main__': block in the Python script, and using/suggesting example_data/* if run from there. But I can add that myself next time I work on this. (Probably that'll be Monday.)
I left one comment inline, otherwise the changes look OK. I didn't test.
I would appreciate if you kept the module self-contained, which would mean keeping the
if __name__ == '__main__':
block in the Python script, and using/suggestingexample_data/*
if run from there. But I can add that myself next time I work on this. (Probably that'll be Monday.)I left one comment inline, otherwise the changes look OK. I didn't test.