Opened 7 years ago
Closed 7 years ago
Last modified 7 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)
Change History (9)
comment:1 Changed 7 years ago by
comment:3 Changed 7 years ago by
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 7 years ago by
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 7 years ago by
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 7 years ago by
Attachment: | nuv_planar_audio.patch added |
---|
comment:6 Changed 7 years ago by
Thanks Gregorio.
jya: the attached patch seems to fix my recordings, could you review?
comment:7 Changed 7 years ago by
I have it fixed, I just need to push whenever I have some bandwidth...
comment:8 Changed 7 years ago by
Resolution: | → Fixed |
---|---|
Status: | new → closed |
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
Can you try after reverting: 9ec7421138adef73ef9cb83b74e046092fecdd6a
and report?