Skip to content

RDKEMW-13310: Adding Telemetry markers in Player-Interface component #82

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
dp0000 wants to merge 57 commits into
base: develop
Choose a base branch
from
Open

RDKEMW-13310: Adding Telemetry markers in Player-Interface component #82

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
57 commits
Select commit Hold shift + click to select a range
313035d
RDKEMW-13310: Telemetry T2 markers for Player-interface component
Deepikasri1234 Feb 17, 2026
beae9af
T2 changes
Deepikasri1234 Feb 18, 2026
71a8b63
resovling compilation issues
Deepikasri1234 Feb 19, 2026
b9d1c24
underflow msg added
Deepikasri1234 Feb 19, 2026
7c20fb0
static code usage is removed
Deepikasri1234 Feb 19, 2026
aaa77d4
Merge branch 'develop' into feature/RDKEMW-13310
Deepikasri1234 Feb 23, 2026
64366e1
resolving conflicts
Deepikasri1234 Feb 23, 2026
f404e6e
added error PTS conditions
Deepikasri1234 Mar 4, 2026
7339125
Merge branch 'feature/dev_sprint_pli' into feature/RDKEMW-13310
Deepikasri1234 Mar 4, 2026
e4d686e
resolved compilation isseus
Deepikasri1234 Mar 4, 2026
9148d7e
resolved compilation issues
Deepikasri1234 Mar 5, 2026
b0ace39
Merge branch 'feature/dev_sprint_pli' into feature/RDKEMW-13310
Deepikasri1234 Mar 6, 2026
1692915
removed additional logic handled
Deepikasri1234 Mar 6, 2026
3a28ef6
added prints
Deepikasri1234 Mar 6, 2026
0ccd026
fixed compilation issues
Deepikasri1234 Mar 6, 2026
6107254
Merge branch 'feature/dev_sprint_pli' into feature/RDKEMW-13310
Deepikasri1234 Mar 9, 2026
b67c5a6
Merge branch 'feature/dev_sprint_pli' into feature/RDKEMW-13310
Deepikasri1234 Mar 11, 2026
3c90d8a
added debug
Deepikasri1234 Mar 12, 2026
fe2013c
Merge branch 'feature/dev_sprint_pli' into feature/RDKEMW-13310
Deepikasri1234 Mar 12, 2026
16117e1
added few telemetry markers
Deepikasri1234 Mar 25, 2026
4f6db42
added new set of markers
Deepikasri1234 Mar 31, 2026
c13064e
Merge branch 'feature/dev_sprint_pli' into feature/RDKEMW-13310
Deepikasri1234 Mar 31, 2026
24076a3
Merge branch 'feature/dev_sprint_pli' into feature/RDKEMW-13310
dp0000 Jun 2, 2026
24b2f08
added t2_event_d commands
Deepikasri1234 Jun 4, 2026
1de869e
Update PlayerTelemetry.h
dp0000 Jun 5, 2026
4769534
Update PlayerTelemetry.h
dp0000 Jun 5, 2026
627c301
added compilation issue fix
Deepikasri1234 Jun 5, 2026
de2bf29
added com fix
Deepikasri1234 Jun 5, 2026
b3b6a2f
added com fix
Deepikasri1234 Jun 5, 2026
80f07bb
com issue fixed
Deepikasri1234 Jun 5, 2026
0254603
speeling check
Deepikasri1234 Jun 5, 2026
0f1e364
spell check
Deepikasri1234 Jun 5, 2026
a8130c4
Merge branch 'develop' into feature/RDKEMW-13310
dp0000 Jul 1, 2026
4db99b2
resolving compilation failures
Deepikasri1234 Jul 2, 2026
a210add
Update PlayerTelemetry.h
dp0000 Jul 3, 2026
2a06647
Update InterfacePlayerRDK.cpp
dp0000 Jul 3, 2026
51e4139
removed entry markers
Deepikasri1234 Jul 15, 2026
df904f5
telemetry marker verification
Deepikasri1234 Jul 15, 2026
3b02f23
Update TelemetryMarkers.h
dp0000 Jul 15, 2026
c807cf2
added initial prints
Deepikasri1234 Jul 16, 2026
daf6ce8
Update InterfacePlayerRDK.cpp
dp0000 Jul 16, 2026
5373b66
Update opencdmsessionadapter.cpp
dp0000 Jul 20, 2026
0a1d146
Update opencdmsessionadapter.cpp
dp0000 Jul 20, 2026
35ccd29
Potential fix for pull request finding
dp0000 Jul 20, 2026
e55bf1a
Potential fix for pull request finding
dp0000 Jul 20, 2026
ba6fe0a
Potential fix for pull request finding
dp0000 Jul 20, 2026
881c70a
Update InterfacePlayerRDK.cpp
dp0000 Jul 20, 2026
c0ae9f4
Merge branch 'develop' into feature/RDKEMW-13310
dp0000 Jul 20, 2026
4e25993
Potential fix for pull request finding
dp0000 Jul 20, 2026
4d5d38b
added few markers
Deepikasri1234 Jul 20, 2026
2fae7ff
added headers
Deepikasri1234 Jul 24, 2026
502e85b
added markers for drm mismatch
Deepikasri1234 Jul 24, 2026
d90ea3a
relevant headers added
Deepikasri1234 Jul 24, 2026
7ad007a
Update DrmSessionManager.cpp
dp0000 Aug 5, 2026
74a4b25
Merge branch 'develop' into feature/RDKEMW-13310
dp0000 Aug 5, 2026
71bbb77
added telemetry for watermarking scenario
Deepikasri1234 Aug 5, 2026
983b43f
initialisation missed
Deepikasri1234 Aug 5, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,12 @@ if(CMAKE_SYSTEM_NAME STREQUAL "Darwin")
else()
set(LIBPLAYERGSTINTERFACE_DEPENDS ${OS_LD_FLAGS} ${UUID_LINK_LIBRARIES} ${LIBCJSON_LINK_LIBRARIES} ${GSTREAMERBASE_LINK_LIBRARIES} ${GSTREAMER_LINK_LIBRARIES} ${CURL_LINK_LIBRARIES} ${LIBDASH_LINK_LIBRARIES} ${LibXml2_LIBRARIES} ${CMAKE_THREAD_LIBS_INIT} ${OPENSSL_LIBRARIES} ${OPENGL_LIBRARIES} ${GLEW_LIBRARIES} -ldl)
endif()

if(CMAKE_TELEMETRY_2_0_REQUIRED)
message("CMAKE_TELEMETRY_2_0_REQUIRED set")
set(LIBPLAYERGSTINTERFACE_SOURCES "${LIBPLAYERGSTINTERFACE_SOURCES}" PlayerTelemetry2.cpp)
set(LIBPLAYERGSTINTERFACE_DEFINES "${LIBPLAYERGSTINTERFACE_DEFINES} -DPLAYER_TELEMETRY_SUPPORT=1")
set(LIBPLAYERGSTINTERFACE_DEPENDS ${LIBPLAYERGSTINTERFACE_DEPENDS} "-ltelemetry_msgsender")
Comment on lines +151 to +153

Copilot AI Mar 6, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Appending PlayerTelemetry2.cpp to LIBPLAYERGSTINTERFACE_SOURCES via set(LIBPLAYERGSTINTERFACE_SOURCES "${LIBPLAYERGSTINTERFACE_SOURCES}" PlayerTelemetry2.cpp) can accidentally turn a list into a single semicolon-delimited string and behaves differently from other source additions in this CMakeLists. Prefer list(APPEND LIBPLAYERGSTINTERFACE_SOURCES PlayerTelemetry2.cpp) (or unquoted ${LIBPLAYERGSTINTERFACE_SOURCES}) for consistent list handling.

Suggested change
set(LIBPLAYERGSTINTERFACE_SOURCES "${LIBPLAYERGSTINTERFACE_SOURCES}" PlayerTelemetry2.cpp)
set(LIBPLAYERGSTINTERFACE_DEFINES "${LIBPLAYERGSTINTERFACE_DEFINES} -DPLAYER_TELEMETRY_SUPPORT=1")
set(LIBPLAYERGSTINTERFACE_DEPENDS ${LIBPLAYERGSTINTERFACE_DEPENDS} "-ltelemetry_msgsender")
list(APPEND LIBPLAYERGSTINTERFACE_SOURCES PlayerTelemetry2.cpp)
set(LIBPLAYERGSTINTERFACE_DEFINES ${LIBPLAYERGSTINTERFACE_DEFINES} -DPLAYER_TELEMETRY_SUPPORT=1)
set(LIBPLAYERGSTINTERFACE_DEPENDS ${LIBPLAYERGSTINTERFACE_DEPENDS} -ltelemetry_msgsender)

Copilot uses AI. Check for mistakes.
Comment on lines +149 to +153
endif()
Comment on lines +149 to +154
Comment on lines +149 to +154
set(LIBPLAYERGSTINTERFACE_HEADERS
closedcaptions/CCTrackInfo.h
GstUtils.h
Expand Down
Loading
Loading