Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Re-Add "processed" folder mode #89

Closed

Conversation

alexbelgium
Copy link

Hi, is there an interest to add the option to restore the Processed folder mode, so to store wave files in a temp folder to allow analysis ?

I'll add it to my addon but wonder if there was a wider interest.

Impact on performance (average of 10 analysis on rpi4) :

  • Disabled (PROCESSED_SIZE missing or =0) : 1.93s
  • Enabled (PROCESSED_SIZE=15) : 2.18s

@Nachtzuster
Copy link
Owner

I'm not clear on the use-case: to harvest (interesting?) wav files?

@alexbelgium
Copy link
Author

alexbelgium commented Jun 5, 2024

It stemmed from a birdcall that I heard during the night, and saw on the spectrogram, that was not identified. I would have wanted a way to save the wav file to perform an offline analysis on the birdnet analyzer and understand why : unknown type of call ; threshold too high...

I initially thought about making a button below the spectrogram that would have allowed to save the wav being analyzed, but it didn't seem technically feasible as analysis only takes 2 seconds and we don't know if the detection will work or not.

Then I noticed that birdnet.conf references this "Processed" folder that existed in the original repo but I guess you removed to improve analysis speed. So I thought that it might be useful, when training the system on a new environment, to access the 15 or 30 last wav files and be able to manually analyze them to understand what is happening, while having the possibility to disable it afterwards to keep the analyzer max speed.

So i'm quite ambivalent : indeed the user case is quite niche (therefore being disabled by default) but it can be useful for people who are listening for a strange bird and curious why it didn't register.

And that's why I didn't initially pushed the PR and just put this feature on my addon : I'm not sure most people would be interested, and everything that we add to the analyzer brings risks to timing & stability.

@alexbelgium alexbelgium closed this Jun 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants