Skip to content

Conversation

@mutherr
Copy link

@mutherr mutherr commented Sep 24, 2024

I noticed there wasn't a way to run the model on a gpu, so added a flag to set the device for the pipeline.

@rwetzold
Copy link

rwetzold commented Oct 11, 2024

This is great! It will throw an error on my side though if run without the --gpu flag

UnboundLocalError: cannot access local variable 'device' where it is not associated with a value

@mutherr
Copy link
Author

mutherr commented Oct 11, 2024

Resolved!

@rwetzold
Copy link

Works nicely! Now we just need to get this merged somehow

@mutherr
Copy link
Author

mutherr commented Oct 11, 2024

It says something about workflows requiring approval. Maybe related to merging?

@mutherr
Copy link
Author

mutherr commented Oct 11, 2024

Oh, no you might have to merge it, since you have write access.

@rwetzold
Copy link

I don't, we need @simonw

kokogamedev added a commit to kokogamedev/blip-caption that referenced this pull request Dec 7, 2024
Based on 
simonw#8

Added --gpu flag to run model on a gpu - modification of the blip_caption.py file
kokogamedev added a commit to kokogamedev/blip-caption that referenced this pull request Dec 7, 2024
Based on 
simonw#8

Added --gpu flag to run model on a gpu - modification of the README.md file
@ecentinela
Copy link

@simonw is this going to be merged?

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