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

Add 'audio_delay' FX #1481

Merged
merged 8 commits into from
Jan 25, 2021
Merged

Add 'audio_delay' FX #1481

merged 8 commits into from
Jan 25, 2021

Conversation

mondeja
Copy link
Collaborator

@mondeja mondeja commented Jan 22, 2021

This is a common effect working with audio. Keep in mind that this effect does not fit a loop FX behaviour, since the sound is multidimensional because you can have multiples copies playing at the same time. Even with an offset attribute in current loop FX this effect would be impossible to achieve.

  • I have provided code that clearly demonstrates the bug and that only works correctly when applying this fix
  • I have added suitable tests demonstrating a fixed bug or new/changed feature to the test suite in tests/
  • I have properly documented new or changed features in the documentation or in the docstrings
  • I have properly explained unusual or unexpected code in the comments around it
  • I have formatted my code using black -t py36

@mondeja mondeja added audio Related to AudioClip, or handling of audio in general. feature New addition to the API i.e. a new class, method or parameter. fx Related to advanced effects applied via clip.fx(), or effects in general. labels Jan 22, 2021
@coveralls
Copy link

coveralls commented Jan 22, 2021

Coverage Status

Coverage increased (+0.09%) to 67.401% when pulling dacb02a on mondeja:audio_delay-fx into b519687 on Zulko:master.

@tburrows13
Copy link
Collaborator

I'm not sure about this one... isn't it basically the same as #1358? I wonder if it would make more sense to implement this one in loop, if you think that the decay would work there?

@mondeja
Copy link
Collaborator Author

mondeja commented Jan 22, 2021

I'm not sure about this one... isn't it basically the same as #1358? I wonder if it would make more sense to implement this one in loop, if you think that the decay would work there?

Keep in mind that is not the same as a loop. In a loop, a whole clip is played and after that starts another copy. In a delay effect, you can have multiple copies playing at the same time.

@mondeja mondeja marked this pull request as draft January 22, 2021 00:58
@tburrows13
Copy link
Collaborator

Keep in mind that is not the same as a loop. In a loop, a whole clip is played and after that starts another copy. In a delay effect, you can have multiple copies playing at the same time.

Oh right, in which case this is fine. There would be no way to implement loop with #1358 that allows a 'negative' gap, so this is a good way to do it.

I have a feeling that the description or name could be improved, but I can't take that further right now. Depending on how those change, perhaps the default decay should be 1?

@mondeja
Copy link
Collaborator Author

mondeja commented Jan 22, 2021

I have a feeling that the description or name could be improved

The description sure, but the name is a standard in music production and sound treatment, it feel to me that is the right name.

@mondeja mondeja marked this pull request as ready for review January 22, 2021 10:57
@mondeja mondeja mentioned this pull request Jan 22, 2021
@mondeja mondeja merged commit 6e99653 into Zulko:master Jan 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
audio Related to AudioClip, or handling of audio in general. feature New addition to the API i.e. a new class, method or parameter. fx Related to advanced effects applied via clip.fx(), or effects in general.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants