Skip to content

Iot integration in video streaming#195

Merged
memnochproxy merged 2 commits intogreearb:masterfrom
goyalsaurabh06:video_streaming_iot
Dec 15, 2025
Merged

Iot integration in video streaming#195
memnochproxy merged 2 commits intogreearb:masterfrom
goyalsaurabh06:video_streaming_iot

Conversation

@sivakondri-CT
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Collaborator

@smileyrekiere smileyrekiere left a comment

Choose a reason for hiding this comment

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

See the example uses a dash url, is there information on how this is created so that someone may replicate the test, or a confluence

@sivakondri-CT
Copy link
Copy Markdown
Contributor Author

See the example uses a dash url, is there information on how this is created so that someone may replicate the test, or a confluence
these are the open source dash url's we are using
Example Reference : https://ottverse.com/free-mpeg-dash-mpd-manifest-example-test-urls/

json.dump(data, file, indent=4)


def duration_to_seconds(duration):
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This the duration_to_seconds() duplicates LFCliBase::parse_time(), why not call parse_time?

Copy link
Copy Markdown
Collaborator

@memnochproxy memnochproxy left a comment

Choose a reason for hiding this comment

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

Please use LFCliBase::parse_time().

thread = threading.Thread(target=trigger_iot, args=(iot_ip, iot_port, iot_iterations, iot_delay, iot_device_list, iot_testname, iot_increment))
thread.start()
else:
total_secs = duration_to_seconds(args.duration)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please call parse_time() here

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@memnochproxy ,Thanks for the suggestion! I’ve updated the PR to use LFCliBase.parse_time()

… running the video streaming test

Signed-off-by: sivakondri-CT <kondru.sankar@candelatech.com>
…s into video streaming report

VERIFIED CLI 1 : python3 -u lf_interop_video_streaming.py --mgr 192.168.242.2 \
                                                          --duration 1m \
                                                          --url https://dash.akamaized.net/akamai/bbb_30fps/bbb_30fps.mpd \
                                                          --test_name videostreaming \
                                                          --webgui_incremental 1 \
                                                          --media_source dash \
                                                          --media_quality 4k \
                                                          --postcleanup \
                                                          --precleanup \
                                                          --iot_test \
                                                          --iot_testname "IotVideoStreaming"
VERIFIED CLI 2 : python3 -u lf_interop_video_streaming.py --mgr 192.168.242.2 \
                                                          --duration 1m \
                                                          --url https://dash.akamaized.net/akamai/bbb_30fps/bbb_30fps.mpd \
                                                          --test_name videostreaming \
                                                          --webgui_incremental 1 \
                                                          --media_source dash \
                                                          --media_quality 4k \
                                                          --postcleanup \
                                                          --precleanup

Signed-off-by: sivakondri-CT <kondru.sankar@candelatech.com>
Copy link
Copy Markdown
Collaborator

@memnochproxy memnochproxy left a comment

Choose a reason for hiding this comment

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

Looks fine.

@memnochproxy memnochproxy merged commit 2732ada into greearb:master Dec 15, 2025
2 checks passed
Copy link
Copy Markdown
Collaborator

@memnochproxy memnochproxy left a comment

Choose a reason for hiding this comment

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

Looks fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants