forked from huggingface/transformers
-
Notifications
You must be signed in to change notification settings - Fork 0
Imagebind hf changes #2
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
Merged
EduardoPach
merged 21 commits into
EduardoPach:adding-imagebind
from
RUFFY-369:imagebind_hf
Sep 1, 2024
Merged
Changes from all commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
a9a5539
chore:add func and classes to get vid clips from user given paths
RUFFY-369 d1c33d0
chore:update uniform_chunk_sampling()
RUFFY-369 53fe080
chore:change chunk duration val and type
RUFFY-369 99306ab
chore:update uniform_temporal_subsample()
RUFFY-369 082be8b
chore:update video transforms and few nits
RUFFY-369 1d6c4ea
fix:bug in image processor call on video paths
RUFFY-369 64d6c38
chore:revert to original to test for unmatched outputs
RUFFY-369 558f544
chore:make transformers compliant and few nits
RUFFY-369 9314a57
style:make fixup
RUFFY-369 79c4089
fix:make fix copies
RUFFY-369 f64778d
chore:resolve necessary conflicts
RUFFY-369 02cb2ab
Merge remote-tracking branch 'imagebind/adding-imagebind' into imageb…
RUFFY-369 4d0edbf
resolve merge/change conflicts by pull
RUFFY-369 bc8821f
chore:make everything similar about files
RUFFY-369 fbbb108
test:add image processor tests
RUFFY-369 4099c8c
fix:failing image processor tests
RUFFY-369 2d4cb59
chore:add contributor name for video output matching and image proces…
RUFFY-369 a283626
test:add Processor kwargs and its test
RUFFY-369 04a9e07
fix:ProcessorTesterMixin test failures
RUFFY-369 4b7f5a8
fix:test failure for len of input ids
RUFFY-369 e2f3064
chore:add custom image and audio kwargs class and some nits
RUFFY-369 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 hidden or 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 hidden or 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 hidden or 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.
I believe we need to create custom
AudioKwargsandImageKwargsas we have arguments that are not in the default classesThere 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.
Yeah, need to discuss the custom kwargs so left them out. i will push those changes