Context
Discovered during Course Builder demo video generation. Despite configuring manim.quality: 1080p30 in docgen.yaml, all Manim scenes rendered at 720p30.
Problem
The manim_runner.py quality flag mapping (_quality_flag) only includes:
"480p15": "-ql",
"720p30": "-qm",
"1080p60": "-qh",
"2160p60": "-qp",
When 1080p30 is specified, it doesn't match any key and falls back to the default medium quality (-qm = 720p30). There is no warning that the requested quality was not honored.
Expected behavior
1080p30 should map to Manim's --resolution 1920,1080 --frame_rate 30 flags, or at minimum, docgen manim should print a warning like:
WARNING: quality '1080p30' not recognized, falling back to 720p30.
Valid options: 480p15, 720p30, 1080p60, 2160p60
Recommendations
Severity
Medium — the pipeline completes but produces lower quality output than configured. Combined with issue #3 lesson 4 (720p is too blurry for text), this silently degrades the final video.
Context
Discovered during Course Builder demo video generation. Despite configuring
manim.quality: 1080p30indocgen.yaml, all Manim scenes rendered at 720p30.Problem
The
manim_runner.pyquality flag mapping (_quality_flag) only includes:When
1080p30is specified, it doesn't match any key and falls back to the default medium quality (-qm= 720p30). There is no warning that the requested quality was not honored.Expected behavior
1080p30should map to Manim's--resolution 1920,1080 --frame_rate 30flags, or at minimum,docgen manimshould print a warning like:Recommendations
_quality_flagto cover common resolutions:1080p30,1080p60,1440p30,1440p601080p30: usemanim render --resolution 1920,1080 --frame_rate 30instead of the preset quality flags{width}p{fps}and pass explicit--resolutionand--frame_rateflags[manim] Rendering at 1080p30 (-qh --frame_rate 30)Severity
Medium — the pipeline completes but produces lower quality output than configured. Combined with issue #3 lesson 4 (720p is too blurry for text), this silently degrades the final video.