-
Notifications
You must be signed in to change notification settings - Fork 28.2k
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.
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
🚨 Add Blip2ForImageTextRetrieval #29261
Merged
amyeroberts
merged 38 commits into
huggingface:main
from
jpizarrom:add_blip2_image_text_retrieval_model
Aug 27, 2024
Merged
Changes from 23 commits
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
e106340
add Blip2ForImageTextRetrieval
jpizarrom 164d824
use one line and remove unnecessary space in tests
jpizarrom 715f6fa
use value from the config, rather than hardcoded
jpizarrom da0cc83
change order of params in Blip2QFormerModel.forward
jpizarrom 02a0e08
update docstring
jpizarrom 360f537
fix style
jpizarrom 5e7764f
update test_inference_opt
jpizarrom 7e7135a
move embeddings out of Blip2QFormerModel
jpizarrom 258f349
remove from_vision_qformer_configs
jpizarrom cf42e9b
remove autocast float16 in Blip2QFormerModel
jpizarrom 8216085
rename fiels into vision_projection,text_projection,use_image_text_ma…
jpizarrom e2781cb
use CLIPOutput for Blip2ImageTextMatchingModelOutput
jpizarrom 1c9746d
remove past_key_values_length from Blip2TextEmbeddings
jpizarrom e6da638
fix small typo in the CLIPOutput docstring
jpizarrom c65ea33
add Blip2ForImageTextRetrieval to Zero Shot Image Classification mapping
jpizarrom a39f9fd
Merge remote-tracking branch 'upstream/main' into add_blip2_image_tex…
jpizarrom a2b99a9
Merge remote-tracking branch 'upstream/main' into add_blip2_image_tex…
jpizarrom 6468947
Merge remote-tracking branch 'upstream/main' into add_blip2_image_tex…
jpizarrom 9bda979
update docstring and add require_torch_fp16
jpizarrom efa8041
rollback test_inference_opt
jpizarrom cb7beae
use use_image_text_matching_head=True in convert
jpizarrom 05cb8d4
Merge remote-tracking branch 'upstream/main' into add_blip2_image_tex…
jpizarrom 8eca5e5
skip test_model_get_set_embeddings
jpizarrom e31b7e5
fix create_rename_keys error on new itm fields
jpizarrom 8808b8a
revert to do scale after dot product between "query" and "key"
jpizarrom 0a72567
fix ValueError on convert script for blip2-opt-2.7b
jpizarrom 2534ca3
update org of paths to Salesforce
jpizarrom af53840
Merge remote-tracking branch 'upstream/main' into add_blip2_image_tex…
jpizarrom f841c59
add is_pipeline_test_to_skip for VisualQuestionAnsweringPipelineTests
jpizarrom 1720ba4
[run_slow] blip_2
jpizarrom 11f0579
removed Blip2ForImageTextRetrieval from IGNORE_NON_AUTO_CONFIGURED
jpizarrom 5e63cdb
fix docstring of Blip2ImageTextMatchingModelOutput
jpizarrom 95232f8
[run_slow] blip_2
jpizarrom 04bb860
fix multi-gpu tests
jpizarrom 5c66e50
[run_slow] blip_2
jpizarrom 2419c85
Merge remote-tracking branch 'upstream/main' into add_blip2_image_tex…
jpizarrom 53abc06
Merge remote-tracking branch 'upstream/main' into add_blip2_image_tex…
jpizarrom d32e889
[run_slow] blip_2
jpizarrom File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Making this optional is a bit funny given the name of the method. We should at least update the docstring to indicate that language model config is optional.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
docstring were updated