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.

Sign up for GitHub

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

Jump to bottom

SD deinterlace issue? #108

Open
9000h opened this issue Mar 15, 2018 · 6 comments
Open

SD deinterlace issue? #108

9000h opened this issue Mar 15, 2018 · 6 comments

Comments

@9000h
Copy link
Contributor

9000h commented Mar 15, 2018

can someone confirm the bad deinterlace issue on:

EURONEWS GERMAN SD;Globecast:12226:HC34M2S0:S19.2E:27500:2440=2:2445=deu@3:0:0:31230:1:1091:0

with the old softhddevice it's perfectly fine but did log
[softhddev] too short PES video packet
last: 3
last: 3
last: 3
last: 3
@9000h
Copy link
Contributor Author

9000h commented Mar 16, 2018 •
edited
Loading

it's probably not deinterlacing
and a warning did show up on the console
WARNING: codec layer incorrectly fills in MPEG-2 slice_vertical_position. Workaround applied

screenshot from 2018-03-16 16-43-59

@9000h
Copy link
Contributor Author

9000h commented Mar 16, 2018 •
edited
Loading

I have also channels where some other logs showing up in the console

iFILM English OBSOLETE;OBSOLETE IRIB:12437:HC34M2O20P0S1:S13.0E:29900:1330=27:1331=eng@4,1332=per@4:0:0:7933:318:8700:0

WARNING: invalid surface provided for second field
WARNING: !!!incorrect slice_param. The first_mb_in_slice of next_slice is less than or equal to that in current slice
vdr: gen8_mfd.c:558: gen8_mfd_avc_directmode_state: Assertion `obj_surface->bo && obj_surface->private_data' failed.

pesintta added a commit that referenced this issue Mar 17, 2018
@pesintta
Move surface synchronization and check sync status
677c8c8 
Move surface synchronization into separate function and utilize that to make
sure deinterlacer references are ready to be used.

Also check status of surface synchronization and log errors in debug log.

This can help with issue #108
@pesintta
Copy link
Owner

I found one possible reason for the message WARNING: invalid surface provided for second field and pushed new code to get rid of that.

@9000h
Copy link
Contributor Author

9000h commented Mar 17, 2018

unfortunately it did not help against the distortions, but the warning seems to be gone

@9000h
Copy link
Contributor Author

9000h commented Mar 17, 2018 •
edited
Loading

maybe the issue is still there I got again the warning switching channels
WARNING: invalid surface provided for second field

@9000h
Copy link
Contributor Author

9000h commented Apr 16, 2018

the master did show the same issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@9000h @pesintta