Skip to content
This repository was archived by the owner on Jan 26, 2023. It is now read-only.
/ rdk-aamp Public archive

Commit ad84341

Browse files
Anurag KrishnanPaulpandian Mariappan
Anurag Krishnan
authored and
Paulpandian Mariappan
committed
DELIA-57156:Junk values in tune parameters affecting tune metrics data
Reason for change: The issue happens because the audio fragment download is finished after video first frame logging.Fix added to calculate the decodestreambucket value based only on video fragments Test Procedure: Check tune metrics Risks: Low Signed-off-by: Anurag Krishnan <[email protected]> Change-Id: I47f43482eae2940ab18c23478a2aee05d173da4c (cherry picked from commit 3fb41b7)
1 parent 37d79c3 commit ad84341

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

AampProfiler.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -250,7 +250,7 @@ void ProfileEventAAMP::GetClassicTuneTimeInfo(bool success, int tuneRetries, int
250250
xreTimeBuckets[TuneTimePlay] = tuneStartMonotonicBase + MAX(buckets[PROFILE_BUCKET_MANIFEST].tFinish, MAX(buckets[PROFILE_BUCKET_PLAYLIST_VIDEO].tFinish, buckets[PROFILE_BUCKET_PLAYLIST_AUDIO].tFinish));
251251
xreTimeBuckets[TuneTimeDrmReady] = MAX(xreTimeBuckets[TuneTimePlay], (tuneStartMonotonicBase + buckets[PROFILE_BUCKET_LA_TOTAL].tFinish));
252252
long long fragmentReadyTime = tuneStartMonotonicBase + MAX(buckets[PROFILE_BUCKET_FRAGMENT_VIDEO].tFinish, buckets[PROFILE_BUCKET_FRAGMENT_AUDIO].tFinish);
253-
xreTimeBuckets[TuneTimeStartStream] = MAX(xreTimeBuckets[TuneTimeDrmReady],fragmentReadyTime);
253+
xreTimeBuckets[TuneTimeStartStream] = MAX(xreTimeBuckets[TuneTimeDrmReady], (tuneStartMonotonicBase + buckets[PROFILE_BUCKET_FRAGMENT_VIDEO].tFinish));
254254
xreTimeBuckets[TuneTimeStreaming] = tuneStartMonotonicBase + buckets[PROFILE_BUCKET_FIRST_FRAME].tStart;
255255

256256
unsigned int failRetryBucketTime = tuneStartMonotonicBase - playerLoadTime;

0 commit comments

Comments
 (0)