cp: fix(deps): clear wheel-bundled ffmpeg CVEs from the all image (part 1/2) (2192) into r1.3.0#2196
cp: fix(deps): clear wheel-bundled ffmpeg CVEs from the all image (part 1/2) (2192) into r1.3.0#2196svcnvidia-nemo-ci wants to merge 1 commit into
fix(deps): clear wheel-bundled ffmpeg CVEs from the all image (part 1/2) (2192) into r1.3.0#2196Conversation
…/2) (#2192) * fix(deps): clear wheel-bundled ffmpeg CVEs from the all image (part 1/2) Signed-off-by: Dong Hyuk Chang <9426164+thomasdhc@users.noreply.github.com> Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * fix(video): adapt motion-vector backend to PyAV 17 API Signed-off-by: Dong Hyuk Chang <9426164+thomasdhc@users.noreply.github.com> Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Signed-off-by: Dong Hyuk Chang <9426164+thomasdhc@users.noreply.github.com> Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: NeMo Bot <nemo-bot@nvidia.com>
|
/ok to test 870c6ad |
Greptile SummaryThis PR changes the video and interleaved dependency set. The main changes are:
Confidence Score: 4/5The changed video decode dependencies need fixes before merging.
nemo_curator/stages/video/filtering/motion_vector_backend.py, pyproject.toml Important Files Changed
|
| ctx = stream.codec_context | ||
| # Set this flag to return motion vectors | ||
| ctx.flags2 |= av.codec.context.Flags2.EXPORT_MVS | ||
| ctx.flags2 |= av.codec.context.Flags2.export_mvs |
There was a problem hiding this comment.
Real decode_for_motion() calls reach this line before any frames are decoded, but PyAV exposes export_mvs as the codec-context flag attribute while the Flags2 enum member is EXPORT_MVS. If PyAV 17.1.0 does not define a lowercase class member, this raises AttributeError at runtime and the current tests miss it because they mock the AV stack.
| "nemo_curator[vllm]", | ||
| "cvcuda_cu12", | ||
| "pycuda", | ||
| "PyNvVideoCodec==2.0.2; (platform_machine == 'x86_64' and platform_system != 'Darwin')", | ||
| "torch<=2.10.0", | ||
| "torchaudio", | ||
| ] |
There was a problem hiding this comment.
Existing x86_64 Linux users who install nemo_curator[video_cuda12] no longer get PyNvVideoCodec, but direct NV codec paths such as the stitching decoder still instantiate VideoBatchDecoder and raise when that package is missing. This changes an established CUDA video install from working hardware decode to a runtime crash unless users discover and install the new video_media extra.
beep boop [🤖]: Hi @thomasdhc 👋,