Skip to content
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

idk if this works, but here's a small clip implementation #79

Open
wants to merge 7 commits into
base: dev
Choose a base branch
from

Conversation

korakoe
Copy link
Contributor

@korakoe korakoe commented Oct 9, 2023

No description provided.

@korakoe
Copy link
Contributor Author

korakoe commented Oct 9, 2023

I have to redownload my dataset so idk if it works

swapping to HF clip
this took way too long, on the plus side, we can also do clip with more than 77 tokens
also now doing image generation sequentially because adding clip to inference is harder than adding it to training
@korakoe
Copy link
Contributor Author

korakoe commented Oct 10, 2023

THIS IS VERY BROKEN AND DOES NOT WORK BECAUSE IM NOT SMART ENOUGH

(works for training, not for inference, if someone wants to take a look at my poorly written code then why not)

@korakoe
Copy link
Contributor Author

korakoe commented Oct 10, 2023

The main reason this doesnt work is because idk how to do negative embeddings here, nor do I know how to filter the logits from clip

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.

1 participant