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

fixed small error in the global threshold calculation #221

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bnestor
Copy link
Contributor

@bnestor bnestor commented Jan 5, 2025

Addresses #219

The number of positive calls had to be greater than the min_num_positive_calls_threshold to trigger a global prediction

@bnestor bnestor added the bug Something isn't working label Jan 5, 2025
@bnestor bnestor requested review from micya and akashmjn as code owners January 5, 2025 23:00
@micya micya requested a review from pastorep January 6, 2025 00:02
@pastorep
Copy link
Collaborator

pastorep commented Feb 4, 2025

I'm a bit confused about the impact of this bug (which I think is secondary to the issue you raise here): #219 (comment).

Semantically, I understand that "minimum" suggests inclusivity. Meaning, if min positive call == 1 and I have one positive call, I should meet that threshold.

In the referenced thread, you mention that you had 15 seconds of high SNR (signal-noise ratio) in a global 15 second timeframe. In that case, the model detected only one individual positive call for the full 15 seconds and didn't meet the "true minimum" of 2 (i.e. no second call because the first call took full 15 seconds)? Is this the correct understanding?

@bnestor
Copy link
Contributor Author

bnestor commented Feb 7, 2025

My model uses 15-second subsamples of the 1-minute audio files. The local time-frame infers on 15 second chunks instead of 2-second chunks in the fastai model. So in my global model, I only need 1 of the local time-frames to be positive.

local_predictions_15s_windows_15s_hoplength=[0,1,0,0] # should be a global prediction of 1, but is currently 0
local_predictions_2s_windows_2s_hoplength=[0,0,0,0,0,0,0,0,1,0,1,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0] # practically no impact on the current model behaviour, especially when the hop length is less than the window length.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants