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

Added DialoGPT Small and Large Models #317

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

Conversation

copoer
Copy link

@copoer copoer commented Jan 1, 2023

In order to make the smaller and larger more easy to access I have added them.
WIP: I am currently in the process of uploading the rust models to hugging face

@copoer
Copy link
Author

copoer commented Jan 1, 2023

Pull requests for added models:
https://huggingface.co/microsoft/DialoGPT-small/discussions/1
https://huggingface.co/microsoft/DialoGPT-large/discussions/4

Waiting for added models to be approved

* Testing model

* Added model config, vocab, merges

* Revert testing
@guillaume-be
Copy link
Owner

Thank you @copoer , please let me know when the model files get merged.

@diptanu
Copy link

diptanu commented Apr 23, 2023

@guillaume-be one idea here could be having an organization for this community on Huggingface where users could upload the rust models, that way you could merge PRs like this faster and switch to official repos once the PRs are merged.

@guillaume-be
Copy link
Owner

Hello @diptanu ,

I agree it is possible to create a new model card without having to contact the original authors - the preferred way would still be to add Rust-compatible model weights on the original model card to avoid licensing issues and to improve discoverability of the Rust-based implementation/avoid duplicate model pages.

Unfortunately it seems the original authors seldom get notifications of PR on the model hub since a lot of community PR are left opened for a long time

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