-
-
Notifications
You must be signed in to change notification settings - Fork 192
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 gain peak #200
base: main
Are you sure you want to change the base?
Add gain peak #200
Conversation
This needs default parameters, so I can do Also, I tried to illustrate some examples of gain curves (green) that I imagine this class could generate and apply in the future (when it is in a more finished state): One thing to note from this illustration is that I imagine that it could use an offset, so that the peak can actually appear anywhere, even before the start or after the end of the audio By the way, I'm curious, what was your motivation for starting to make this class? Are you working on some audio AI application that could benefit from this transform? |
Judging my your pics, you suggest using As for motivation - I think this class can be beneficial for some audio AI tasks like recognition of badly received signals (for example, signal you received has some louder parts, and some more quiet parts). |
Yeah, take the gain curve with a grain of salt, especially the min gain part :P I guess min gain isn't the most important feature here, but the difference between min gain and max gain. |
Yeah, the offset can be selected randomly |
OK, I'll modify the code accordingly. |
Sweet, thanks 👍 |
As for offset, I think that one can additionally use |
Good question! Let's consider the possibilities:
So in order to offer more variability in the results, I still think it's better that the gain peak itself gets an offset, instead of relying on |
|
Can you reply? I'll modify my code more if necessary. |
Hi :) Thanks for the effort so far, and thanks for the patience. I've been in crunch mode at work for the past few days. I saw that there are at least these two things that I would like to check before merge:
I'll try to give it a look at some point in the coming days |
Nice! And thanks for your reply. |
Thanks for the patience. This is still on my TODO list 🙈 |
I had another look at this now, and gathered some thoughts. Here's what I imagine would be good to have in/for this transform:
I imagine that it would also be nice if it could invert its behavior, so it'll essentially be a gain dip instead of a gain peak. Maybe this can be achieved with the gain diff parameter. At the moment there's quite a gap between what I imagine/desire and what is coded thus far in this pull request. Here's what I propose: I make a GainPeak some time later. When I start doing that, I'll close this PR and make my own branch where I cherry-pick your commits into it, so you'll be listed as a contributor, and then I'll try to implement all the features I suggested above. |
OK, let's do as you suggest. |
Attempt to implement #181.