-
Notifications
You must be signed in to change notification settings - Fork 205
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
Add configuration options for sox spectrogram creation #19
Open
hoover67
wants to merge
2,420
commits into
kahst:main
Choose a base branch
from
mcguirepr89:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
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
Uwe, are you sure you want to merge this into the BirdNET-Lite repo ? Or did you mean the BirdNET-Pi repo by Patrick ? |
Yep, it was meant for Patrick's repo but my github-fu is too weak to create a proper pull request there, so sorry for the trouble and bothering you. I created a comment / discussion over at Patrick's repo outlining the changes. All the best, Uwe |
ehpersonal38
force-pushed
the
main
branch
7 times, most recently
from
March 7, 2023 23:55
ec5a8a0
to
4de5618
Compare
Co-Authored-By: Datore7 <[email protected]>
Added net-tools to the needed packages.
Update install_services.sh
Co-Authored-By: gutocarv <[email protected]>
Simplify handle_client loop / indent, check for EOF when reading from…
recordings, not necessarily the root filesystem
check for its own 'success' state and don't try to validate the numbers
…end up with low / no disk space and be unable to "escape." In the event of a backlog, the loop in birdnet_analysis.sh runs continuously, which means analyzed files do not get moved, which means in turn extract_new_birdsounds.sh cannot create mp3 extractions and move the processed files again, which would allow them to be cleaned up by the cron jobs. This patch is the obvious fix, i.e. to just limit the maximum amount of files birdnet_analysis.sh can process to 20. This should be a good trade off given the small overhead of listing files etc before the main loop begins - note that for the "normal" case of no backlog, birdnet_analysis usually exits after a single analysis anyway.
Chunk analysis calls to facilitate recovery from stalls & low disk space
Fix GeoIP data validation
Check disk usage of filesystem hosting recordings
adding Rwanda to list of countries
and Omar, Belarus, and the Czech Republic
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.
This patch will add the possibility to add further arguments to sox' spectrogram creation, for instance to create grayscale images, change the spectrogram's size or use a different DFT window algorithm.
Right now this needs to be configured in /etc/birdnet/birdnet.conf using a shell variable, no GUI config possible yet. :-(
sox.zip
All the best, Uwe
Affected files:
o scripts/spectrogram.sh
o scripts/extract_new_birdsongs.sh
o birdnet.conf-defaults
EDIT: I had to put the tgz into a zip file because github refused the upload. Sorry for any trouble, it's the first time I contribute to a project using github :-)