Skip to content

Fix parent classes of ProcessingKwargs#40676

Merged
zucchini-nlp merged 2 commits intohuggingface:mainfrom
cyyever:fix_processing_kwargs
Sep 5, 2025
Merged

Fix parent classes of ProcessingKwargs#40676
zucchini-nlp merged 2 commits intohuggingface:mainfrom
cyyever:fix_processing_kwargs

Conversation

@cyyever
Copy link
Copy Markdown
Contributor

@cyyever cyyever commented Sep 4, 2025

What does this PR do?

The parent classes of ProcessingKwargs look incorrect.

Signed-off-by: cyy <cyyever@outlook.com>
@cyyever cyyever force-pushed the fix_processing_kwargs branch from 3ce630c to 2c156f8 Compare September 4, 2025 06:53
@cyyever cyyever changed the title FIx parent classes of ProcessingKwargs Fix parent classes of ProcessingKwargs Sep 4, 2025
@Rocketknight1
Copy link
Copy Markdown
Member

cc @zucchini-nlp here too!

Copy link
Copy Markdown
Member

@zucchini-nlp zucchini-nlp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks reasonable to me since we already have it as images/text/etc_kwargs and the current way is redundant. I don't think it will break anything, let's merge it

@zucchini-nlp zucchini-nlp enabled auto-merge (squash) September 5, 2025 09:52
@zucchini-nlp zucchini-nlp merged commit 93df343 into huggingface:main Sep 5, 2025
23 checks passed
@HuggingFaceDocBuilderDev
Copy link
Copy Markdown

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.

@cyyever cyyever deleted the fix_processing_kwargs branch September 6, 2025 06:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants