Add ray data option for video benchmarks#2002
Conversation
Signed-off-by: Onur Yilmaz <oyilmaz@nvidia.com>
Greptile SummaryThis PR adds
Confidence Score: 5/5Config-only addition that faithfully mirrors existing xenna entries; no logic changes. All four raydata entries are verified to be exact copies of their xenna counterparts with only --executor=ray_data substituted. GPU resource flags, argument sets, timeouts, clip count requirements, and throughput thresholds are all consistent across every pair. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[nightly-benchmark.yaml] --> B[video_pipeline_benchmark.py]
B --> C{--executor}
C -->|xenna| D[video_embedding_xenna]
C -->|ray_data| E[video_embedding_raydata]
C -->|xenna| F[video_transcoding_xenna]
C -->|ray_data| G[video_transcoding_raydata]
C -->|xenna| H[video_captioning_xenna]
C -->|ray_data| I[video_captioning_raydata]
C -->|xenna| J[video_transnetv2_..._xenna]
C -->|ray_data| K[video_transnetv2_..._raydata]
Reviews (1): Last reviewed commit: "Add ray data option for video benchmarks" | Re-trigger Greptile |
| - metric: num_clips_generated | ||
| exact_value: 1400 | ||
| - metric: throughput_clips_per_sec | ||
| min_value: 4.0 | ||
|
|
There was a problem hiding this comment.
The throughput might be different for ray data. Do we want to run verify and then add these or add these first and then fix?
Add ray_data variants for video nightly benchmarks
Summary
Adds ray_data executor coverage for the four video pipelines that currently only run on Xenna in nightly benchmarks, so both backends are tracked side-by-side every night — matching the dual-executor pattern already used for
audio_readspeech_*,image_curation_*,domain_classification_*, etc.Config-only change to benchmarking/nightly-benchmark.yaml — no Python edits needed because video_pipeline_benchmark.py already accepts
--executor={xenna,ray_data}and routes through setup_executor().Changes
Renamed each existing video benchmark to add an explicit
_xennasuffix and added a sibling_raydataentry (identical args except--executor=ray_data):num_clipsvideo_embedding_xennavideo_embedding_raydatavideo_transcoding_xennavideo_transcoding_raydatavideo_captioning_xennavideo_captioning_raydatavideo_transnetv2_motion_aesthetic_filter_embeddings_xennavideo_transnetv2_motion_aesthetic_filter_embeddings_raydata