Birdnet integration #112
Merged
Birdnet integration #112
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Added BirdNet Pipeline
- Upgrades to tensorflow 2.8.0 which is compatible with both Microfaune and BirdNET
birdnet "model" isolation parameter was mispelled as "birndet"
…into birdnet_integration
Updated example to include the "model" isolation parameter
currently calls generate_automated_labels on the single clip - will refactor in a different commit to handle if an already made automated_df is passed in
- Made sure it is in the conda_environments folder
- Changed local_score_visualization() to spectrogram_visualization() - Changed plot_bird_label_scores() to binary_visualization() - Fixed a small bug in spectrogram_visualization() related to an error warning when a dataframe with automated annotations from multiple clips is passed in
JacobGlennAyers
requested review from
sprestrelski,
shreyasar2202 and
nishantbalaji
- Fixed a small bug in spectrogram_visualization() to better handle both bools and dataframes for automated_df - Added spectrogram_graph(), annotation_duration_histogram() to README
There was a problem hiding this comment.
There are a couple of things that I noticed
- The BirdNet README needs to be updated with the new argument that we added.
- The color codes look confusing for this cell when using BirdNet isolation parameters
The markdown cell mentions white for true negatives, but the spectrogram shows blue
Apart from these, the changes look good to me!
|
Shreyas, when you say argument added, which function are you referring to? |
I am referring to the 'num_predictions' command line argument |
Cool, just changed our repo |
- Some markdown didn't make sense talking about TPs, FPs, FNs, and TNs in the context of the spectrogram_visualization() changes
…into birdnet_integration
This was
linked to
issues
May 23, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.