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 xlm-roberta-large-xnli model card #6723

Merged
merged 3 commits into from
Aug 26, 2020
Merged

add xlm-roberta-large-xnli model card #6723

merged 3 commits into from
Aug 26, 2020

Conversation

joeddav
Copy link
Contributor

@joeddav joeddav commented Aug 25, 2020

No description provided.

@joeddav joeddav added the model card Related to pretrained model cards label Aug 25, 2020
@joeddav joeddav requested a review from julien-c August 25, 2020 16:13
@codecov
Copy link

codecov bot commented Aug 25, 2020 •
edited
Loading

Codecov Report

Merging #6723 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6723      +/-   ##
==========================================
- Coverage   79.00%   78.96%   -0.05%     
==========================================
  Files         156      156              
  Lines       28405    28405              
==========================================
- Hits        22442    22429      -13     
- Misses       5963     5976      +13
Impacted Files Coverage Δ
src/transformers/generation_tf_utils.py 80.70% <0.00%> (-3.01%) ⬇️
src/transformers/file_utils.py 82.41% <0.00%> (-0.26%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a25c9fc...715d491. Read the comment docs.

@joeddav joeddav merged commit 99407f9 into huggingface:master Aug 26, 2020
@joeddav
Copy link
Contributor Author

joeddav commented Aug 26, 2020

Merging so I can tweet, but lmk if anything is off and I'll update it.

@julien-c
Copy link
Member

looks good! Fixed tiny typos in 3242e4d

@joeddav joeddav deleted the xlm-roberta-large-xnli-card branch August 28, 2020 15:10
@samsontmr
Copy link

Hi @joeddav! Could you share your hyperparameters for training the model (I assume you used the run_glue)? Would you say that the last phase of training helped significantly?

Zigur pushed a commit to Zigur/transformers that referenced this pull request Oct 26, 2020
@joeddav @Zigur
add xlm-roberta-large-xnli model card (huggingface#6723)
59c80fa 
* add xlm-roberta-large-xnli model card

* update pt example

* typo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@julien-c julien-c Awaiting requested review from julien-c

Assignees
No one assigned
Labels
model card Related to pretrained model cards
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@joeddav @julien-c @samsontmr