Repository navigation
Modular conversion small fix - #48934
Conversation
|
[For maintainers] Suggested jobs to run (before merge) run-slow: gemma4_unified |
| @@ -1185,6 +1185,8 @@ def replace_class_node( | |||
| "ImageProcessorPil": "image_processing_pil", | |||
| "VideoProcessor": "video_processing", | |||
| "VideoProcessorInitKwargs": "video_processing", | |||
| "VideoProcessorKwargs": "video_processing", | |||
| "ImageProcessorPilKwargs": "image_processing_pil", | |||
There was a problem hiding this comment.
don't ask me why 😆
noticed in another internal release and if 100% honest, we should have never had two TypedDicts defined in each backend...
My long due plan to delete that and fix import/modular issues in centralized way
vasqu
left a comment
vasqu
left a comment
There was a problem hiding this comment.
Thanks lol, what a weird interaction here. Feel like ive seen the vid processor kwargs stuff somewhere else at some point, might be worth to sanity check based on some regex pattern or claude/codex
|
i know right, i can't remember where but i defi made the fix at some point in some PR |
CI recapDashboard: View test results in Grafana |
|
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
What does this PR do?
as per title