Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

docs: update names of open clip models #580

Merged
merged 5 commits into from
Oct 19, 2022
Merged

Conversation

guenthermi
Copy link
Member

@guenthermi guenthermi commented Oct 18, 2022

Update names of open clip models

We are going to change the the separator in the names of the open clip models from # to ::

  • This PR references an open issue
  • I have added a line about this change to CHANGELOG

Copy link
Member

@bwanglzu bwanglzu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Member

@gmastrapas gmastrapas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Member

@scott-martens scott-martens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume the list is accurate, so LGTM.

@github-actions
Copy link

📝 Docs are deployed on https://ft-docs-update-open-clip-names--jina-docs.netlify.app 🎉

@bwanglzu bwanglzu merged commit 46f8541 into main Oct 19, 2022
@bwanglzu bwanglzu deleted the docs-update-open-clip-names branch October 19, 2022 13:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants