Skip to content

Conversation

isaac-chung
Copy link
Collaborator

@isaac-chung isaac-chung commented Jun 23, 2025

Fixes #2771

If you add a model or a dataset, please add the corresponding checklist:

@isaac-chung isaac-chung linked an issue Jun 23, 2025 that may be closed by this pull request
@@ -38,15 +41,42 @@ def __init__(

# Load model
self.mdl = model_class.from_pretrained(
model_name, revision=revision, device_map=self.device, **kwargs
model_name, device_map=self.device, **kwargs
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

With revision there, I kept running into this error, which disappeared when I removed revision:

File "/opt/homebrew/anaconda3/envs/mteb/lib/python3.11/site-packages/transformers/modeling_utils.py", line 1100, in _get_resolved_checkpoint_files raise EnvironmentError( OSError: vidore/colpaligemma-3b-mix-448-base does not appear to have a file named pytorch_model.bin, model.safetensors, tf_model.h5, model.ckpt or flax_model.msgpack.

Copy link
Member

@Samoed Samoed Jun 23, 2025

Choose a reason for hiding this comment

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

I don't think that we want to remove revision here. Which model did you use?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I tested all 3. Could you see if you can run any of these successfully with revision?

mteb run -m vidore/colpali-v1.2 -t MNIST

@isaac-chung isaac-chung marked this pull request as ready for review June 23, 2025 12:49
@isaac-chung isaac-chung requested a review from Samoed June 23, 2025 12:49
@isaac-chung isaac-chung merged commit 277ab49 into v2.0.0 Jun 25, 2025
9 checks passed
@isaac-chung isaac-chung deleted the copali-new-interface-v2 branch June 25, 2025 06:52
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.

v2 Refactor colpali modles to new interface
2 participants