Fix missing arg in check_docstring

[yonigozlan]

What does this PR do?

As the title says

[HuggingFaceDocBuilderDev]

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.

[zucchini-nlp]

I saw your reply about this new kwarg under a different PR, so I will move my q here for easier communication
Do we need to add it as a default kwarg for all image processors? IMO only a few VLMs where the image_seq_length is saved in config should have it in model's image processor.

Otherwise LGTM, just want to clarify before merging

[yonigozlan]

@zucchini-nlp Fully agreed I'll try to think of something better. The difficulty I found is that some models on the hub use this attribute in their image processor, even if there's no mention of it anywhere in the corresponding model files. So it's hard to determine which model should support this kwarg.

In the meantime though, this missing arg is causing a lot of annoying warning (totally my fault) so if we can merge this quickly that'd be great.

[zucchini-nlp]

Yep, also experienced the warnings today 😄 Oke, let's merge and clean-up later by adding to the image processors where it is indeed needed/used

IIRC these are now accessed by processor like self.image_processor.image_seq_length, so we can search models with this pattern