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

Allow Clips providers to be used for validation #40

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

puddly
Copy link
Contributor

@puddly puddly commented Dec 21, 2024

This PR allows for training/testing and ambient validation using just clips, without pre-generating the features. It replaces the split_count parameter:

-split_count=0.1,
+random_splits={
+   'training': 0.8,
+   'testing': 0.1,
+   'validation': 0.1,
+   'testing_ambient': 0.0,
+   'validation_ambient': 0.0,
+},

My goal was to replace the pre-generated negative spectrogram features with just WAV files (to test changing clip_duration_ms).

The obvious downside of configuring the notebook to not use memory mapped features is that it's way slower than precomputing them once. But it does let you generate an unlimited number of augmentations from a single sample, which might help with generalizability.

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.

1 participant