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

Forest IR Synthesis #157

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Forest IR Synthesis #157

wants to merge 10 commits into from

Conversation

TQZhang04
Copy link

Added a Pytorch module to synthesize forest reverb into the augmentations file
Imported the necessary files from https://github.com/microsoft/Forest_IR_synthesis/tree/main
(My auto formatter also made a bunch of tiny style changes to the other code within augmentations.py, but it shouldn't change function)

TQ Zhang and others added 10 commits January 22, 2024 16:26
changed the import statements in the forestIR files to specify pyha_analyzer
Added the reverb module to the data aug pipeline

BREAKING CHANGE: dataset, test now import all functions from augmentations
changed AddReverb to simulate a set number of impulses, then randomly pick from those to convolve with
removed prints
commented out assert statements in Reverb to (hopefully) speed up code
@Sean1572
Copy link
Collaborator

@TQZhang04 Any updates for this project? I'd still be interested in experimenting with this once we get the testing datasets

@TQZhang04
Copy link
Author

I agree, haven't really touched this since winter '24 but this was one of the data aug methods I wanted to revisit

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.

2 participants