search

janclemenslab / das

Deep Audio Segmenter
http://janclemenslab.org/das/
Apache License 2.0
28 stars 10 forks source link

Predict error #10

Closed emorysaba closed 3 years ago

emorysaba commented 3 years ago

Hello! I'm running into an issue with the predict function (screenshot attached below) - I've tried re-assembling datasets and re-training models every which way and still run into this issue about 2/3rds of the time when using it. Any idea as to what could be causing this? Thank you in advance!

Screen Shot 2021-07-08 at 5 53 18 PM
postpop commented 3 years ago

Hi,

thank you for raising this issue! We pad the signal to fill batches at the end of the signal - this can lead to cases where there are detections that extend beyond the original signal. I will add a check that prevents that from happening and will let you know once that is fixed.

postpop commented 3 years ago

I pushed a potential fix. Can you update with pip install das --upgrade and let me know whether this fixes the issue? Thanks!

emorysaba commented 3 years ago

I downloaded the new version from the GitHub page and unfortunately, there seems to be a new error - now whenever I load any audio file, this pops up

Screen Shot 2021-07-09 at 12 01 27 PM
emorysaba commented 3 years ago

These are the errors when opening the file and running predict, respectively in full

Screen Shot 2021-07-09 at 12 03 55 PM Screen Shot 2021-07-09 at 12 04 02 PM
postpop commented 3 years ago

I've pushed a new version that should fix the predict issue. Please upgrade the GUI with pip install xarray-behave --upgrade and let me know whether this helps.

The other error is independent - I've opened a new issue #11. Looks like some issue caused by a new version of pyqtgraph or qt. Could you post the output of conda list there? Sorry about the hassle...

emorysaba commented 3 years ago

Hi! The original problem seems to be fixed, but now when I run predict, the model finds 1 or no segments where it used to accurately predict many.

postpop commented 3 years ago

I messed this up, terribly sorry about that. I've pushed code that fixes the issue now. Hopefully... Please update das via pip install das --upgrade and let me know whether all works as expected now. Thanks for your help and your patience!

emorysaba commented 3 years ago

Yes, it seems to be working as expected - thank you for all your help, I really do appreciate it!

postpop commented 3 years ago

Great - closing this.