Skip to content

Conversation

@stas00
Copy link
Contributor

@stas00 stas00 commented Sep 16, 2020

As discussed at #6940 (comment)
fixing to use the correct decorator add_start_docstrings

@sgugger

@codecov
Copy link

codecov bot commented Sep 16, 2020

Codecov Report

Merging #7174 into master will decrease coverage by 0.40%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #7174      +/-   ##
==========================================
- Coverage   80.71%   80.31%   -0.41%     
==========================================
  Files         169      169              
  Lines       32293    32293              
==========================================
- Hits        26066    25936     -130     
- Misses       6227     6357     +130     
Impacted Files Coverage Δ
src/transformers/tokenization_marian.py 99.15% <100.00%> (ø)
src/transformers/tokenization_mbart.py 96.82% <100.00%> (ø)
src/transformers/tokenization_pegasus.py 95.23% <100.00%> (ø)
src/transformers/modeling_tf_electra.py 25.32% <0.00%> (-73.63%) ⬇️
src/transformers/configuration_bart.py 90.00% <0.00%> (-4.00%) ⬇️
src/transformers/data/data_collator.py 92.83% <0.00%> (-0.72%) ⬇️
src/transformers/modeling_tf_utils.py 87.01% <0.00%> (-0.33%) ⬇️
src/transformers/modeling_bart.py 93.77% <0.00%> (-0.17%) ⬇️
src/transformers/tokenization_dpr.py 57.65% <0.00%> (+4.50%) ⬆️
src/transformers/modeling_mobilebert.py 89.45% <0.00%> (+10.24%) ⬆️
... and 2 more

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 df16506...e73ba1f. Read the comment docs.

@sgugger sgugger merged commit 42049b8 into huggingface:master Sep 16, 2020
@stas00 stas00 deleted the addstr branch September 16, 2020 18:46
Zigur pushed a commit to Zigur/transformers that referenced this pull request Oct 26, 2020
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.

2 participants