From 486474bd67006f401fd24c96b5d2dfea5dab8df7 Mon Sep 17 00:00:00 2001 From: CarolinePascal Date: Mon, 4 May 2026 17:01:11 +0200 Subject: [PATCH] chore(doctrings): updating docstrings --- src/lerobot/datasets/video_utils.py | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/src/lerobot/datasets/video_utils.py b/src/lerobot/datasets/video_utils.py index 244c7d8c3..8ba9c856e 100644 --- a/src/lerobot/datasets/video_utils.py +++ b/src/lerobot/datasets/video_utils.py @@ -115,11 +115,12 @@ class VideoEncoderConfig: check_video_encoder_config_pyav(self) def resolve_vcodec(self) -> None: - """Validate vcodec and resolve 'auto' to best available HW encoder, fallback to libsvtav1. + """Check ``vcodec`` and, when it is ``"auto"``, pick a concrete encoder. - Any explicitly-requested codec that isn't in the local FFmpeg build is - also silently rewritten to ``libsvtav1`` so encoding never hard-fails on - a host missing the requested encoder. + For ``"auto"``, the first hardware encoder in the preference list that FFmpeg + exposes is chosen; if none are available, ``libsvtav1`` is used. If the + resolved codec (explicit or after auto-selection) is not present in the + local FFmpeg build, raises ``ValueError``. """ if self.vcodec not in VALID_VIDEO_CODECS: raise ValueError(f"Invalid vcodec '{self.vcodec}'. Must be one of: {sorted(VALID_VIDEO_CODECS)}") @@ -130,7 +131,7 @@ class VideoEncoderConfig: logger.info(f"Auto-selected video codec: {encoder}") self.vcodec = encoder return - logger.info("No hardware encoder available, falling back to software encoder 'libsvtav1'") + logger.warning("No hardware encoder available, falling back to software encoder 'libsvtav1'") self.vcodec = "libsvtav1" if self.detect_available_encoders(self.vcodec):