Skip to content

Add support for CoOp models to ClipDetection #373

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
wants to merge 26 commits into
base: develop
Choose a base branch
from
Open

Add support for CoOp models to ClipDetection #373

wants to merge 26 commits into from

Conversation

ZachCafego
Copy link
Contributor

@ZachCafego ZachCafego commented Jun 6, 2024 •
edited by jrobble
Loading

ZachCafego and others added 19 commits May 3, 2024 13:55
@ZachCafego
Current iteration which runs CoOp evaluation
88310b7
@ZachCafego
Removed local directories
5992eca
@ZachCafego
Component now runs CoOp and returns MPF Image Location objects.
a909e5a
@ZachCafego
Implemented self.trainer, and removed excess comments. Probably need …
2619a37 
…to add more descriptive logging for our purposes.
@ZachCafego
Component is now self-contained with CoOp.
95f76b0
@ZachCafego
Updated README and LICENSE files.
0f0b6df
@ZachCafego
Listed modifications to CoOp repo for use in the component.
9b58335
@ZachCafego
Listed modifications for clip.py and listed modified files in MODIFIC…
08d99d8 
…ATIONS.
@ZachCafego
Temporarily added test_video_file_coop unittest to show that CoOp wor…
111b389 
…ks with batched images.
@ZachCafego
Removed unnecessary files
b01d05c
@ZachCafego
Component builds on Docker and completes image runs with CoOp on CPU …
21859f1 
…and GPU
Added CUDA_DEVICE_ID property.
790eef7
Added support for CUDA_DEVICE_ID property
d8dc7a2
@ZachCafego
Fixed nested quotes issue.
41f61f5
@ZachCafego
Cleaned up file. Both image and video files can be run through CoOp.
cafec45
@ZachCafego
CUDA support for CoOp fixed. CoOp args are written to file and read f…
a4c5ab0 
…or each run.
@ZachCafego
Moved model loading to CoOpWrapper init. Moved CoOp unittests off of …
e68c07a 
…test_clip.py.
@ZachCafego
Removed references to private testing files.
a4273fc
@ZachCafego
Removed unused import and fixed issue where python instance was creat…
fc0c238 
…ed on wrong GPU for inferencing.
@ZachCafego ZachCafego requested a review from jrobble June 6, 2024 20:44
@ZachCafego ZachCafego self-assigned this Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jrobble jrobble Awaiting requested review from jrobble

At least 1 approving review is required to merge this pull request.

Assignees

@ZachCafego ZachCafego

Labels

enhancement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@ZachCafego