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

Also download new weights #49

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Also download new weights #49

wants to merge 2 commits into from

Conversation

Flova
Copy link
Member

@Flova Flova commented Aug 13, 2022

Proposed changes

Add reference to the new anchorless weights

@Flova Flova requested a review from jaagut August 13, 2022 19:32
@phinik
Copy link
Member

phinik commented Dec 19, 2023

Maybe we should also put the pretrained encoder weights on data.bit-bots and add it to the download file (if there are no legal issues) ?

@Flova
Copy link
Member Author

Flova commented Dec 22, 2023

What is the issue with downloading them from the official source?

@Flova
Copy link
Member Author

Flova commented Dec 22, 2023

Ah this pr here also has the yoeox cfg file weird.

@phinik
Copy link
Member

phinik commented Dec 22, 2023

What is the issue with downloading them from the official source?

We haven't documented where to download them from... Maybe you remember that I could not find the correct file back then, and the one I had found online did not work despite having the correct name.

It would make things much easier if we include them in the download_weights script and let the script download them either from data.bit-bots or the official source (didn't know there was such a thing) -- whatever you prefer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🔖 Ready
Development

Successfully merging this pull request may close these issues.

2 participants