Duration based Audio Error Concealment - Change Default

Looks good.

=============================================
Eric Hyche (ehychereal.com)
Technical Lead
RealNetworks, Inc.  

> -----Original Message-----
> From: datatype-dev-bounceshelixcommunity.org 
> [mailto:datatype-dev-bounceshelixcommunity.org] On
Behalf Of 
> ext-anuj.dhamijanokia.com
> Sent: Wednesday, April 16, 2008 3:34 PM
> To: 'datatype-devhelixcommunity.org'
> Cc: 'nokia-private-dev-bounceshelixcommunity.org'
> Subject: [datatype-dev] Duration based Audio Error 
> Concealment - ChangeDefault
> 
> 
> 			"Nokia submits this code under the 
> terms of a commercial contribution agreement with 
> RealNetworks, and I am authorized to contribute this
code 
> under said agreement."
> 
> 			  
> 			Modified by:  ext-anuj.dhamijanokia.com 
> 			  
> 			Reviewed by: 
> 			  
> 			Date: 04/16/2008 
> 			  
> 			TSW Error ID: EADA-7D5Q5U 
> 
> 			Project: SymbianMmf_rel 
> 			  
> 			Synopsis: Change the Audio Error 
> Concealment Default Duration  
> 
> 			Overview: 
> 			Field Testing found 90 ms to be better 
> fit for most of test cases related to Audio Error 
> Concealment. So change the default value in
Configuration 
> file from 20ms to 90 ms.
> 
> 			Fix: 
> 			Modify variable AudioErrorConcDuration 
> value to 90 (it was 20 before). 
> 
> 
> 			Files modified & changes: 
> 
> 			
>
srcclientappssymbiancommonconfigR1_Mobile_4_0_Factory.cf
g 
> 
> 			Image Size and Heap Use impact: None 
> 
> 			Module Release testing (STIF, Audio) : Passed 
> 
> 			Test case(s) Added  : No 
> 
> 			Memory leak check performed : Passed, 
> No leaks found 
> 			  
> 			Platforms and Profiles Build Verified: 
> helix-client-s60-32-mmf-mdf-arm 
> 
> 			Platforms and Profiles Functionality 
> verified: armv5 
> 			  
> 			Branch: Head, 210CayS & 221CayS 
> 
> 
> 			Index: R1_Mobile_4_0_Factory.cfg 
> 			
>
============================================================
======= 
> 			RCS file: 
>
/cvsroot/clientapps/symbiancommon/config/R1_Mobile_4_0_Facto
ry.cfg,v 
> 			retrieving revision 1.26.2.22.2.2 
> 			diff -u -w -r1.26.2.22.2.2 
> R1_Mobile_4_0_Factory.cfg 
> 			--- R1_Mobile_4_0_Factory.cfg   26 Mar 
> 2008 19:46:15 -0000      1.26.2.22.2.2 
> 			+++ R1_Mobile_4_0_Factory.cfg   16 Apr 
> 2008 18:43:38 -0000 
> 			 -120,5 +120,5  
> 
> 			 #Audio Error Concealment Duration 
> (milliseconds) 
> 			 #In case of packet drop interpolate 
> Audio Packets for this duration 
> 			-#defaults to 20 milliseconds 
> 			-AudioErrorConcDuration=20 
> 			+#defaults to 90 milliseconds 
> 			+AudioErrorConcDuration=90 
> 
> 			thnx & regds 
> 			AD 
> 
> ______________________________________________ 
> From:   Dhamija Anuj
(EXT-InfoVisionConsultants-MSW/Dallas)  
> Sent:   Wednesday, March 26, 2008 12:10 PM 
> To:     'nokia-private-dev-bounceshelixcommunity.org'; 
> 'datatype-devhelixcommunity.org' 
> Subject:        CR: Duration based Audio Error
Concealment 
> 
> 			"Nokia submits this code under the 
> terms of a commercial contribution agreement with 
> RealNetworks, and I am authorized to contribute this
code 
> under said agreement."
> 
> 			  
> 			Modified by:  ext-anuj.dhamijanokia.com 
> 			  
> 			Reviewed by: 
> 			  
> 			Date: 03/24/2008 
> 			  
> 			Project: SymbianMmf_rel 
> 			  
> 			Synopsis: Change the Audio Error 
> Concealment logic to be based on Duration instead of
Number 
> of Packets 
> 
> 			Overview: 
> 			Current Audio Error Concealment 
> involves replicating prior (faded) audio packets for a

> predefined number of packets in case of packet drop.
Modify 
> this to replicate packets for a predefined duration
rather 
> than number of packets. This predefined duration is
defined 
> as a parameter in configuration file. Its default value
is 
> set to 20 milliseconds.
> 
> 			Fix: 
> 			Introduce new class members in 
> HXSymbianSwAudioDecoder for Configured Error
Concealment 
> Duration and Consecutive Loss Duration (Counter). Read
in the 
> Configured value for Error Concealment from
configuration 
> file when Decoder is opened. Every time packet drop
occurs 
> increment Consecutive loss duration by the duration of
the 
> frame and introduce the faded copy of last audio frame

> played. If Consecutive Loss Duration exceeds the
configured 
> Error Concealment duration then insert silence.
> 
> 			Reset the Consecutive Loss Duration to 
> zero in Decode method which is invoked when packets are

> received without any error.
> 
> 			New Field Added to configuration file: 
> AudioErrorConcDuration 
> 
> 			Files modified & changes: 
> 
> 			
>
srcdatatypemp4audiomdfpubplatformsymbiansymbianswdec
oder.h 
> 			
>
srcdatatypemp4audiomdfplatformsymbiansymbianswdecoder
.cpp 
> 
> 			
>
srcclientappssymbiancommonconfigR1_Mobile_4_0_Factory.cf
g 
> 
> 			Image Size and Heap Use impact: None 
> 
> 			Module Release testing (STIF, Audio) : Passed 
> 
> 			Test case(s) Added  : No 
> 
> 			Memory leak check performed : Passed, 
> No leaks found 
> 			  
> 			Platforms and Profiles Build Verified: 
> helix-client-s60-32-mmf-mdf-arm 
> 
> 			Platforms and Profiles Functionality 
> verified: armv5 
> 			  
> 			Branch: Head, 210CayS & 221CayS 
> 
> 
> 			------------ 
> 			thnx & regds 
> 			AD              
> 
> 
> 
> 


_______________________________________________
Datatype-dev mailing list
Datatype-devhelixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/da
tatype-dev