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.

Sign up for GitHub

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

Jump to bottom

Add model card for singbert. #6674

Merged
merged 5 commits into from
Aug 25, 2020
Merged

Add model card for singbert. #6674

merged 5 commits into from
Aug 25, 2020

Conversation

zyuanlim
Copy link
Contributor

Adding a model card for singbert- bert for singlish and manglish.

@zyuanlim
Add model card for singbert.
ad8084b 
Adding a model card for singbert- bert for singlish and manglish.
@zyuanlim
Update README.md
63ac83c 
Add additional tags and model name.
@zyuanlim
Update README.md
fa5079e 
Fix tag for malay.
JetRunner
JetRunner approved these changes Aug 24, 2020
View reviewed changes
Copy link
Contributor

@JetRunner JetRunner left a comment

Choose a reason for hiding this comment

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

This very nice lah! Thanks for ur contribution :)

zyuanlim and others added 2 commits August 25, 2020 07:40
@zyuanlim @JetRunner
Update model_cards/zanelim/singbert/README.md
a6044b9 
Fix language

Co-authored-by: Kevin Canwen Xu <canwenxu@126.com>
@zyuanlim
Add examples and custom widget input.
0cb6a55 
Add examples and custom widget input.
@zyuanlim
Copy link
Contributor Author

@JetRunner Just a final addition of couple of more examples and customized the widget inputs, good to go :)

@zyuanlim
Copy link
Contributor Author

Not sure if I should bring this up here or raise a new issue, but when i tested my widget (hosted inference), i got this error:

Can't load config for 'zanelim/singbert'. Make sure that: - 'zanelim/singbert' is a correct model identifier listed on 'https://huggingface.co/models' - or 'zanelim/singbert' is the correct path to a directory containing a config.json file

However the config file is there as confirmed by

>>> transformers-cli s3 ls
singbert/config.json             2020-08-24T05:54:28.000Z "6004cb287370530537f4076c9cf7fdbe"       471 
singbert/pytorch_model.bin       2020-08-24T05:53:37.000Z "c060a644d84e55b2f93aa67fb1f35956" 440509997 
singbert/special_tokens_map.json 2020-08-24T05:54:23.000Z "8b3fb1023167bb4ab9d70708eb05f6ec"       112 
singbert/tf_model.h5             2020-08-24T05:52:23.000Z "45a8eea544f73079768bb136fe3d0a27" 536061440 
singbert/tokenizer_config.json   2020-08-24T05:54:25.000Z "8b3fb1023167bb4ab9d70708eb05f6ec"       112 
singbert/vocab.txt               2020-08-24T05:53:32.000Z "767659dd848f37f6937a0ffb833ee6b1"    224170

@JetRunner
Copy link
Contributor

Not sure if I should bring this up here or raise a new issue, but when i tested my widget (hosted inference), i got this error:


Can't load config for 'zanelim/singbert'. Make sure that: - 'zanelim/singbert' is a correct model identifier listed on 'https://huggingface.co/models' - or 'zanelim/singbert' is the correct path to a directory containing a config.json file

However the config file is there as confirmed by


>>> transformers-cli s3 ls

singbert/config.json             2020-08-24T05:54:28.000Z "6004cb287370530537f4076c9cf7fdbe"       471 

singbert/pytorch_model.bin       2020-08-24T05:53:37.000Z "c060a644d84e55b2f93aa67fb1f35956" 440509997 

singbert/special_tokens_map.json 2020-08-24T05:54:23.000Z "8b3fb1023167bb4ab9d70708eb05f6ec"       112 

singbert/tf_model.h5             2020-08-24T05:52:23.000Z "45a8eea544f73079768bb136fe3d0a27" 536061440 

singbert/tokenizer_config.json   2020-08-24T05:54:25.000Z "8b3fb1023167bb4ab9d70708eb05f6ec"       112 

singbert/vocab.txt               2020-08-24T05:53:32.000Z "767659dd848f37f6937a0ffb833ee6b1"    224170

Don't worry. It's sometimes flaky. As long as you can load the model with from_pretrained and you are good to go. 😉

@JetRunner JetRunner merged commit b6512d2 into huggingface:master Aug 25, 2020
Zigur pushed a commit to Zigur/transformers that referenced this pull request Oct 26, 2020
@zyuanlim @JetRunner @Zigur
Add model card for singbert. (huggingface#6674)
4e920bf 
* Add model card for singbert.

Adding a model card for singbert- bert for singlish and manglish.

* Update README.md

Add additional tags and model name.

* Update README.md

Fix tag for malay.

* Update model_cards/zanelim/singbert/README.md

Fix language

Co-authored-by: Kevin Canwen Xu <canwenxu@126.com>

* Add examples and custom widget input.

Add examples and custom widget input.

Co-authored-by: Kevin Canwen Xu <canwenxu@126.com>
fabiocapsouza pushed a commit to fabiocapsouza/transformers that referenced this pull request Nov 15, 2020
@zyuanlim @JetRunner @fabiocapsouza
Add model card for singbert. (huggingface#6674)
d6afad4 
* Add model card for singbert.

Adding a model card for singbert- bert for singlish and manglish.

* Update README.md

Add additional tags and model name.

* Update README.md

Fix tag for malay.

* Update model_cards/zanelim/singbert/README.md

Fix language

Co-authored-by: Kevin Canwen Xu <canwenxu@126.com>

* Add examples and custom widget input.

Add examples and custom widget input.

Co-authored-by: Kevin Canwen Xu <canwenxu@126.com>
fabiocapsouza added a commit to fabiocapsouza/transformers that referenced this pull request Nov 15, 2020
@fabiocapsouza
Revert "Add model card for singbert. (huggingface#6674)"
c0139c5 
This reverts commit d6afad4.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JetRunner JetRunner JetRunner approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zyuanlim @JetRunner