Skip to content

[GLM-OCR] GLM-OCR Support #43391

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
zRzRzRzRzRzRzR wants to merge 26 commits into
base: main
Choose a base branch
from
Open

[GLM-OCR] GLM-OCR Support #43391

zRzRzRzRzRzRzR wants to merge 26 commits into from
+3,073 −21

Conversation

@zRzRzRzRzRzRzR
Copy link
Contributor

@zRzRzRzRzRzRzR zRzRzRzRzRzRzR commented Jan 21, 2026

Need reivew for @zucchini-nlp

@zRzRzRzRzRzRzR zRzRzRzRzRzRzR changed the title GLM-OCR support [GLM-OCR] GLM-OSR Test Support Jan 21, 2026
@zRzRzRzRzRzRzR zRzRzRzRzRzRzR marked this pull request as draft January 21, 2026 13:42
@zRzRzRzRzRzRzR zRzRzRzRzRzRzR changed the title [GLM-OCR] GLM-OSR Test Support [GLM-OCR] GLM-OCR Test Support Jan 21, 2026
zucchini-nlp
zucchini-nlp reviewed Jan 22, 2026
View reviewed changes
Copy link
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.

Great use of modular! Only a few comments and I think the PR will be ready to merge! Also, since the model isn't yet released, let's skip slow integration tests with a "reason" so that CI doesn't complain

src/transformers/models/auto/configuration_auto.py
Comment on lines +187 to +189
("glm_ocr", "GlmOcrConfig"),
("glm_ocr_text", "GlmOcrTextConfig"),
("glm_ocr_vision", "GlmOcrVisionConfig"),
Copy link
Member

@zucchini-nlp zucchini-nlp Jan 22, 2026

Choose a reason for hiding this comment

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

note to myself: this is getting too reprtitive with more MM models added. Will try to come up with a cleaner way!

@zRzRzRzRzRzRzR zRzRzRzRzRzRzR mentioned this pull request Jan 22, 2026
Open
zucchini-nlp
zucchini-nlp approved these changes Jan 23, 2026
View reviewed changes
Copy link
Member

@zucchini-nlp zucchini-nlp left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Great, thanks a lot for un-skipping those tests! I will let you fix one last test that is failing (compile_fullgraph) and ask for Arthur's review. Then we can merge

When done fixing, can you also mark PR as ready for review?

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Jan 23, 2026

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.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 24, 2026

View the CircleCI Test Summary for this PR:

https://huggingface.co/spaces/transformers-community/circle-ci-viz?pr=43391&sha=48594f

@github-actions
Copy link
Contributor

github-actions bot commented Jan 25, 2026

[For maintainers] Suggested jobs to run (before merge)

run-slow: auto, glm46v, glm4v, glm4v_moe, glm_ocr

@zRzRzRzRzRzRzR zRzRzRzRzRzRzR marked this pull request as ready for review January 25, 2026 13:16
@zRzRzRzRzRzRzR
Copy link
Contributor Author

zRzRzRzRzRzRzR commented Jan 25, 2026 •
edited
Loading

This PR also removes all restrictions on image sizes smaller than 28 in GLM-V (support in all GLM-V models)

@zRzRzRzRzRzRzR zRzRzRzRzRzRzR changed the title [GLM-OCR] GLM-OCR Test Support [GLM-OCR] GLM-OCR Support Jan 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zucchini-nlp zucchini-nlp zucchini-nlp approved these changes

@ArthurZucker ArthurZucker Awaiting requested review from ArthurZucker

Assignees

No one assigned

Labels

New model

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@zRzRzRzRzRzRzR @HuggingFaceDocBuilderDev @zucchini-nlp @molbap