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

Fix dropped and repeated audio samples for 1001-based standards #1326

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

scriptorian
Copy link
Contributor

Fixes #1227 with further work to ensure that routes work correctly.
Many thanks to ducthiem90 for most of the work!

@Julusian Julusian added this to the v2.3.1 LTS milestone Jul 6, 2020
@sendust
Copy link

sendust commented Jul 7, 2020

I tried this version with HD1080i 59.94 and NTSC configuration.
After short time test, issue #1319 is not appear any more.
(I think #1319 is also related with this fix)
Let me have a long time test.
Thahk you so much~~

@didikunz
Copy link

didikunz commented Feb 8, 2021

Hi all, anything that hinders this pull request to be merged? It stops me from using it for a project. The client says, that he wants to use 2.0.7 until this is merged. So it would be very much appreciated...

@Julusian
Copy link
Member

Julusian commented Feb 8, 2021

I have some concerns that I drafted but never finished writing a while back, so it is waiting on me to finish reviewing it then someone having the time to revisit/fix those issues

@ronag
Copy link
Member

ronag commented Feb 8, 2021

I don't think this is the way to resolve this. I also have some code in a drawer somewhere for this. Will see if I can find it.

@didikunz
Copy link

didikunz commented Feb 8, 2021

Thanks for posting back, I just wondered, because it says "This brach has no conflicts..."

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.

None yet

5 participants