Skip to content

Conversation

@ManuelFay
Copy link
Collaborator

No description provided.

@ManuelFay
Copy link
Collaborator Author

@QuentinJGMace can we merge this ?

Copy link
Collaborator

@QuentinJGMace QuentinJGMace left a comment

Choose a reason for hiding this comment

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

Sure, just need to update docstrings (I can do it if needed @ManuelFay :-)

Comment on lines 24 to 28
class ColQwenOmniRetriever(BaseVisionRetriever):
"""
ColQwen2 retriever that implements the model from "ColPali: Efficient Document Retrieval
with Vision Language Models".
"""
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
class ColQwenOmniRetriever(BaseVisionRetriever):
"""
ColQwen2 retriever that implements the model from "ColPali: Efficient Document Retrieval
with Vision Language Models".
"""
class ColQwenOmniRetriever(BaseVisionRetriever):
"""
ColQwenOmni retriever that implements the model from "ColPali: Efficient Document Retrieval
with Vision Language Models" for a Qwen-2.5 Omni backbone.
"""

Comment on lines 43 to 44
'Install the missing dependencies with `pip install "vidore-benchmark[colpali-engine]"` '
"to use ColQwen2Retriever."
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
'Install the missing dependencies with `pip install "vidore-benchmark[colpali-engine]"` '
"to use ColQwen2Retriever."
'Install the missing dependencies with `pip install "vidore-benchmark[colpali-engine]"` '
"to use ColQwenOmniRetriever."

@QuentinJGMace QuentinJGMace self-requested a review August 4, 2025 11:48
@QuentinJGMace QuentinJGMace merged commit e0885ea into main Aug 4, 2025
5 checks passed
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.

3 participants