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

Punch and roll: tracks are not played in sync. #2162

Closed
Tracked by #2353
DavidBailes opened this issue Nov 19, 2021 · 12 comments · Fixed by #3071
Closed
Tracked by #2353

Punch and roll: tracks are not played in sync. #2162

DavidBailes opened this issue Nov 19, 2021 · 12 comments · Fixed by #3071
Assignees
Labels
bug An error, undesired behaviour, or missed functionality P2 High priority recording regression Bug is a regression against previous version
Milestone

Comments

@DavidBailes
Copy link
Collaborator

DavidBailes commented Nov 19, 2021

Describe the bug
Punch and roll: tracks are not played in sync.

To Reproduce
Steps to reproduce the behavior:

  1. Record yourself counting to ten.
  2. Duplicate the track.
  3. In the first track, click to position the cursor about half way through.
  4. With overdub enabled, press shift + D for punch and roll.
  5. Playback starts at the pre roll point in track 1, but at the cursor in track 2. The playback of the two tracks is out of sync.

Expected behavior
The playback of the tracks should be in sync.

Additional information (please complete the following information):

  • Win 10
  • Audacity 3.1.2

Additional context
This is a regression wrt 3.0.5.
Reported on the Audacity4Blind list.

@Paul-Licameli
Copy link
Contributor

Thank you for finding this.

It's not good, however it won't be a problem for a narrator just laying down one mono track.

@DavidBailes
Copy link
Collaborator Author

@Penikov Penikov added 3 SP bug An error, undesired behaviour, or missed functionality P3 Medium Priority and removed 3 SP labels Nov 22, 2021
@SteveDaulton
Copy link
Member

and another report on the forum here: https://forum.audacityteam.org/viewtopic.php?f=70&t=121679

@jserquera
Copy link

jserquera commented Dec 18, 2021

I teach punch recording and some of my students cannot work with this bug. Next semester I will have to point the new students to this download page:

https://github.com/audacity/audacity/releases/tag/Audacity-3.1.0

Which is an inconvenience.

Higher priority would be appreciated.

@petersampsonaudacity
Copy link

petersampsonaudacity commented Dec 29, 2021

Yet another user has complained about this on the Forum today:
https://forum.audacityteam.org/viewtopic.php?f=70&t=121679&p=442336#p442336

This was logged as an issue against 3.1.2 back in mid-November and was not fixed for 3.1.3.

It would seem to be a serious problem particularly for readers of audio-books and those making podcasts - I really think this should be addressed for the 3.2.0 release, particularly since it is a regression - it used to work properly in 3.0.5 and earlier.

P3 seems a bit of a low priority given that it is a regression.

Note that it is not listed on (but perhaps it should be):
3.2 backlog tracking #2353

@crsib @Paul-Licameli @Penikov

@petersampsonaudacity petersampsonaudacity added the regression Bug is a regression against previous version label Dec 29, 2021
@petersampsonaudacity
Copy link

And another user complaint today from the Forum:
https://forum.audacityteam.org/viewtopic.php?f=47&t=122453&p=442372

@LWinterberg LWinterberg mentioned this issue Jan 3, 2022
21 tasks
@AnitaBats AnitaBats added this to the Audacity 3.2 milestone Jan 13, 2022
@AnitaBats AnitaBats added P2 High priority and removed P3 Medium Priority labels Jan 13, 2022
@petersampsonaudacity
Copy link

@Penikov @LWinterberg @crsib @Paul-Licameli - another complaint about this today on the Forum:

https://forum.audacityteam.org/viewtopic.php?f=70&t=121679

Is there any chance this bug can be addressed and fixed for the upcoming 3.2.0 ?

@SteveDaulton
Copy link
Member

DavidBailes added a commit to DavidBailes/audacity that referenced this issue Jun 14, 2022
I think the problem started in master with commit:30b7882e107c32

Fix:
If there are any preroll tracks, set the start of all the playback tracks to the same value.
@DavidBailes
Copy link
Collaborator Author

@Paul-Licameli . I had a look at this bug before realising that you'd already been assigned to it.

@Paul-Licameli
Copy link
Contributor

Paul-Licameli commented Jun 14, 2022

@Paul-Licameli . I had a look at this bug before realising that you'd already been assigned to it.

I am overwhelmed with many other things. If you are willing to investigate this deeper, please do.

... and now I see you have. Thank you

@petersampsonaudacity
Copy link

Testing on W10 with @DavidBailes branch build: audacity-win-3.2.0-alpha-20220614+564d2a6-x64

David's branch build appears to fix this problem.

Paul-Licameli pushed a commit that referenced this issue Jul 8, 2022
I think the problem started in master with commit:30b7882e107c32

Fix:
If there are any preroll tracks, set the start of all the playback tracks to the same value.
@petersampsonaudacity
Copy link

Testing on W10 with latest master audacity-win-3.2.0-alpha-20220708+811c241-x64-msvc2022

This is mow fixed on W10 with this latest master.

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 P2 High priority recording regression Bug is a regression against previous version
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants