Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#11507 closed Bug Report - General (Fixed)

Audio pitch an octave higher for .nuv mp4 files

Reported by: Jim Stichnoth Owned by: JYA
Priority: minor Milestone: unknown
Component: MythTV - Audio Output Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

All of my .nuv mp4 files play back with audio and video at the correct speed but with the audio pitch transposed an octave higher. (Like when you breathe in helium and talk.) These files originated as PVR-150 mpeg2 recordings that were transcoded to mpeg4. I assume this happened after the most recent ffmpeg resync, but I can't be positive. Running "mythfrontend -v audio --loglevel debug" shows nothing remarkable.

I uploaded my smallest .nuv recording to https://docs.google.com/file/d/0BxETmfuHvGPQMnVpbkMyS1Q2STg/edit?usp=sharing .

Attachments (1)

nuv_planar_audio.patch (1.9 KB) - added by Jim Stichnoth 6 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 6 years ago by JYA

Can you try after reverting: 9ec7421138adef73ef9cb83b74e046092fecdd6a

and report?

comment:2 Changed 6 years ago by JYA

And which version of myth are you using? 0.26 or master?

comment:3 Changed 6 years ago by Jim Stichnoth

Unfortunately, git bisect says

c22bdae1aa8eb761ddcced6b51bdc10297827d02 is the first bad commit
commit c22bdae1aa8eb761ddcced6b51bdc10297827d02
Author: Jean-Yves Avenard <jyavenard@mythtv.org>
Date:   Sun Feb 17 00:07:20 2013 +1100

    FFmpeg resync to release/1.1 branch as of revision 057051b8
    
    Only tested to compile properly on both mac and linux.

This is running master.

Reverting 9ec7421138adef73ef9cb83b74e046092fecdd6a has no effect.

ffplay from the ffmpeg release/1.1 branch plays the sample correctly.

comment:4 Changed 6 years ago by gregorio.gervasio@…

I think NuppelDecoder::GetFrame?() now needs the conversion from planar audio. It can be changed to use AudioOutputUtil::DecodeAudio? (instead of avcodec_decode_audio4 + av_samples_get_buffer_size).

comment:5 Changed 6 years ago by JYA

Thank you very much for the pointer Gregorio.. That's what I was suspecting as it did sounds that way...

But I was looking into AVFD and not NuppelDecoder?...

Changed 6 years ago by Jim Stichnoth

Attachment: nuv_planar_audio.patch added

comment:6 Changed 6 years ago by Jim Stichnoth

Thanks Gregorio.

jya: the attached patch seems to fix my recordings, could you review?

comment:7 Changed 6 years ago by JYA

I have it fixed, I just need to push whenever I have some bandwidth...

comment:8 Changed 6 years ago by JYA

Resolution: Fixed
Status: newclosed

In 709a6f64b86128473c07d939bd29c44bf37b7625/mythtv:

Fix planar audio for nuv container.

Additionally, properly handle errors and potential issues should an AVPacket contains more than one audio frame

Last edited 6 years ago by JYA (previous) (diff)
Note: See TracTickets for help on using tickets.