search

macaodha / batdetect2

Other
53 stars 10 forks source link

Potential fix for issue #20 #21

Closed EvansMike closed 1 year ago

EvansMike commented 1 year ago

Checks for zero length files and skips processing them.

EvansMike commented 1 year ago

The existing test, cli.py will fail on main branch if you create an empty file in example_data/audio/ On Linux do touch example_data/audio/zero.wav It will pass on issue-20 branch.

mbsantiago commented 1 year ago

Hey @EvansMike, it seems some other tests are now failing. I'm sorry I a bit busy at the moment, but could you please check what happened? Thanks for helping.

EvansMike commented 1 year ago

The extra file makes test_list_audio_files() fail at:

assert len(audio_files) == 3

EvansMike commented 1 year ago

This has become more work than I have time to spend on it.