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

Glitch at the end of playback. #2314

Closed
Tracked by #2214
DavidBailes opened this issue Dec 20, 2021 · 6 comments · Fixed by #2331
Closed
Tracked by #2214

Glitch at the end of playback. #2314

DavidBailes opened this issue Dec 20, 2021 · 6 comments · Fixed by #2331
Assignees
Labels
bug An error, undesired behaviour, or missed functionality P3 Medium Priority bugs

Comments

@DavidBailes
Copy link
Collaborator

DavidBailes commented Dec 20, 2021

Describe the bug
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

  1. Generate a few seconds of a tone
  2. select some audio within this
  3. playback
  4. There's a "glitch" at the end

Expected behavior
No glitches!

Screenshots
Audio selected in Audacity:
image

Playback recorded using loopback in Reaper - note spurious audio after the correct audio has been played:
image

Using Audacity 3.1.2, same project, recorded using loopback in Reaper - no spurious audio:
image

Additional information (please complete the following information):

  • Audacity 3.1.3 commit: fa4ef2
  • Windows 10
@DavidBailes
Copy link
Collaborator Author

DavidBailes commented Dec 20, 2021

Using chirp instead of tone to give more info as to what spurious audio is being played:
Audio selected in Audacity:
image

Using Audacity 3.1.3, playback recorded using reaper:
image

Using Audacity 3.1.2, playback recorded using reaper:
image

@Paul-Licameli Paul-Licameli self-assigned this Dec 20, 2021
@Paul-Licameli Paul-Licameli mentioned this issue Dec 20, 2021
11 tasks
@AnitaBats AnitaBats added bug An error, undesired behaviour, or missed functionality P3 Medium Priority bugs labels Dec 20, 2021
@petersampsonaudacity
Copy link

petersampsonaudacity commented Dec 20, 2021

Confirmed on W10 with altest 3.2.0 alpha

And I have a sneaking suspicion it's related to: Play-at -Speed with a selection gets stuck in Play mode #2316

@Paul-Licameli
Copy link
Member

Paul-Licameli commented Dec 20, 2021

Confirmed on W10 with altest 3.2.0 alpha

And I have a sneaking suspicion it's related to: Play-at -Speed with a selection gets stuck in Play mode #2316

No, not related

@Paul-Licameli
Copy link
Member

Paul-Licameli commented Dec 20, 2021

The problem bisects to 81f74f7

I can fix it by increasing the numbers in NewDefaultPlaybackPolicy::SuggestedBufferTimes such as to 0.25, 0.25, 0.5 but this will have the consequence of making the response to the changing slider more laggy.

@DavidBailes
Copy link
Collaborator Author

DavidBailes commented Dec 20, 2021

I can fix it

Please fix it. If a user selects some audio for playback, then that audio should be played back - not the audio and bit of what comes afterwards. This could lead to inaccurately positioned edits.

@Paul-Licameli
Copy link
Member

Paul-Licameli commented Dec 20, 2021

I can fix it

Please fix it. If a user selects some audio for playback, then that audio should be played back - not the audio and bit of what comes afterwards. This could lead to inaccurately positioned edits.

I fear endless rounds of grief if I do it that way, because the complaint about lagginess will predicatably follow. I hope to find a different way that can reconcile both demands but it is not obvious yet.

Adjustable play-at-speed really is getting a total reimplementation so it can also accomodate loop region changes on the fly.

@AnitaBats AnitaBats added this to the Audacity 3.1.3 milestone Dec 21, 2021
@LWinterberg LWinterberg linked a pull request Dec 22, 2021 that will close this issue
6 tasks
@LWinterberg LWinterberg moved this from To do to Reviewer approved in Sprint 10 - Enhancements&Bug fixes Dec 22, 2021
@AnitaBats AnitaBats moved this from Reviewer approved to Ready for QA in Sprint 10 - Enhancements&Bug fixes Dec 22, 2021
@Penikov Penikov moved this from Ready for QA to In QA in Sprint 10 - Enhancements&Bug fixes Dec 22, 2021
Sprint 10 - Enhancements&Bug fixes automation moved this from In QA to Ready for QA Dec 22, 2021
@LWinterberg LWinterberg moved this from Ready for QA to In QA in Sprint 10 - Enhancements&Bug fixes Dec 22, 2021
@Penikov Penikov moved this from In QA to Done in Sprint 10 - Enhancements&Bug fixes Dec 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An error, undesired behaviour, or missed functionality P3 Medium Priority bugs
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

5 participants