Skip to content

Minor changes to #990 (fixed audio glitches caused by previous audio concatenation method) #1177

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

Merged
merged 4 commits into from
Aug 22, 2022
Merged

Minor changes to #990 (fixed audio glitches caused by previous audio concatenation method) #1177

merged 4 commits into from
Aug 22, 2022

Conversation

thomasmny
Copy link
Contributor

@thomasmny thomasmny commented Aug 18, 2022

Description

All credit for this PR goes to @tonywiththetommy

However his PR has been stale for a while and accumulated some merge conflicts so I decided to redo the PR in hope that it can be merged soon.
Also adds silence_duration to the config template which upstream didn't.

Issue Fixes

Superceeds #990

Checklist:

  • I am pushing changes to the develop branch
  • I am using the recommended development environment
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have formatted and linted my code using python-black and pylint
  • I have cleaned up unnecessary files
  • My changes generate no new warnings
  • My changes follow the existing code-style
  • My changes are relevant to the project

Any other information (e.g how to test the changes)

None

@thomasmny thomasmny changed the title Fixed conflicts for #990 (fixed audio glitches caused by previous audio concatenation method) Fix merge conflicts for #990 (fixed audio glitches caused by previous audio concatenation method) Aug 18, 2022
@thomasmny thomasmny marked this pull request as draft August 18, 2022 11:02
@thomasmny thomasmny marked this pull request as ready for review August 18, 2022 11:47
@thomasmny thomasmny changed the title Fix merge conflicts for #990 (fixed audio glitches caused by previous audio concatenation method) Minor changes to #990 (fixed audio glitches caused by previous audio concatenation method) Aug 18, 2022
@JasonLovesDoggo
Copy link
Collaborator

Will review later

Copy link
Collaborator

@JasonLovesDoggo JasonLovesDoggo left a comment

Choose a reason for hiding this comment

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

Everything looks great I just need to test it first

@JasonLovesDoggo
Copy link
Collaborator

works perfectly thank you @Trichtern this is great

@JasonLovesDoggo JasonLovesDoggo merged commit c995811 into elebumm:develop Aug 22, 2022
@thomasmny thomasmny deleted the feat/add-silence-between-clips branch August 22, 2022 07:00
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