Opened 22 months ago

Closed 22 months ago

Last modified 16 months ago

#13169 closed Bug Report - General (fixed)

video playback profile format codec selection field

Reported by: blm-ubunet@… Owned by: Peter Bennett
Priority: minor Milestone: 29.1
Component: MythTV - Video Playback Version: Master Head
Severity: low Keywords: video format codec
Cc: Ticket locked: no

Description

v30-Pre-198-g6646748 added support to use video format (codec) & resolution to control video playback profile.

This only seems to work iff one keyword (h264) is used. If any other text is added before or after 'h264' then this profile fails for h264.

I've tried comma-space & space only delimiting. All fail.

Is the intention to allow multiple keywords ? I assume valid keywords are h264, mpeg2video, mpeg1video, vc1, wmv3 & mpeg4 or similar. All I'm trying to do is set NOT(h265) or NOT(hevc).

Workaround is to setup a profile for each video codec format.

Change History (8)

comment:1 Changed 22 months ago by Peter Bennett

Owner: changed from JYA to Peter Bennett
Status: new → assigned

comment:2 Changed 22 months ago by Peter Bennett <pbennett@…>

Resolution: → fixed
Status: assigned → closed

In 310ae2aa8afb8a55a8a365e4ce1621ef4337718e/mythtv:

Video Display Profile: Fix bug with multiple codec selection

Multiple codecs can be entered, space separated, in a video display profile.
Fix bug where these were not being handled

Fixes #13169

comment:3 Changed 22 months ago by Peter Bennett <pbennett@…>

In 528d01bbc576210f55b3c884d5f99df38ca897ea/mythtv:

Video Display Profile: Fix bug with multiple codec selection

Multiple codecs can be entered, space separated, in a video display profile.
Fix bug where these were not being handled

Fixes #13169

(cherry picked from commit 310ae2aa8afb8a55a8a365e4ce1621ef4337718e)

comment:4 Changed 22 months ago by Peter Bennett

Space separated is the correct way to enter them. This commit fixes the bug.

comment:5 Changed 22 months ago by Peter Bennett

Milestone: needs_triage → 29.1

comment:6 Changed 22 months ago by blm-ubunet@…

Fantastic work. The additional options for video decoder filters are comma separated, why does this have to be different? Is there any chance the video format/codec could contain a space? i.e. hevc (main) etc. Thanks.

comment:7 Changed 22 months ago by Peter Bennett

You can see a full list by running 

ffmpeg -codecs

There are none with spaces, they sometimes have underscores.

comment:8 Changed 16 months ago by Peter Bennett

Owner: changed from Peter Bennett to Peter Bennett
Note: See TracTickets for help on using tickets.