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

added wav2vec2 model, and related conf/utils files. #107

Merged
merged 5 commits into from
Dec 3, 2024

Conversation

danielle-hausler
Copy link
Collaborator

main changes:

  1. added WAV2VEC2 to models.py (+updated conf_dict)
  2. added files_handler to utils so that models for example from [https://github.com/earthspecies/aves] could be loaded.
  3. added custom conf files.

Copy link
Collaborator

@Z30G0D Z30G0D left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool PR!

@@ -0,0 +1,2 @@
# @package _global_
_preprocessors: []
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if there are no preprocessors why is this file needed and not use the defaults?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wav2vec is working on the raw input (the waveform) and the defaults file has a preprocess of the data into a spectrogram.

soundbay/models.py Outdated Show resolved Hide resolved
soundbay/models.py Outdated Show resolved Hide resolved
@Z30G0D Z30G0D added ci-test label to trigger CI flow and removed ci-test label to trigger CI flow labels Dec 2, 2024
@Z30G0D Z30G0D closed this Dec 3, 2024
@Z30G0D Z30G0D reopened this Dec 3, 2024
@Z30G0D Z30G0D removed the ci-test label to trigger CI flow label Dec 3, 2024
@Z30G0D Z30G0D closed this Dec 3, 2024
@Z30G0D Z30G0D reopened this Dec 3, 2024
danielle-hausler and others added 3 commits December 3, 2024 21:34
Co-authored-by: Tomer Nahshon <33577556+Z30G0D@users.noreply.github.com>
@danielle-hausler danielle-hausler added the ci-test label to trigger CI flow label Dec 3, 2024
fixed indentation
@Z30G0D Z30G0D added ci-test label to trigger CI flow and removed ci-test label to trigger CI flow labels Dec 3, 2024
@danielle-hausler danielle-hausler merged commit 08da933 into master Dec 3, 2024
2 checks passed
@danielle-hausler danielle-hausler deleted the feature/wav2vec branch December 3, 2024 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-test label to trigger CI flow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants