Skip to content

Conversation

@patrickvonplaten
Copy link
Contributor

@patrickvonplaten patrickvonplaten commented Mar 26, 2020

  • Copy past from bart docs
  • polish main docs page
  • improve docstring in modeling_t5.py and modeling_tf_t5.py

@codecov-io
Copy link

codecov-io commented Mar 27, 2020

Codecov Report

Merging #3461 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3461   +/-   ##
=======================================
  Coverage   77.79%   77.80%           
=======================================
  Files         100      100           
  Lines       17049    17051    +2     
=======================================
+ Hits        13264    13267    +3     
+ Misses       3785     3784    -1     
Impacted Files Coverage Δ
src/transformers/modeling_bart.py 97.58% <ø> (ø)
src/transformers/tokenization_t5.py 95.83% <ø> (ø)
src/transformers/modeling_t5.py 81.29% <100.00%> (+0.08%) ⬆️
src/transformers/modeling_tf_t5.py 94.98% <100.00%> (ø)
src/transformers/modeling_tf_utils.py 88.32% <0.00%> (+0.17%) ⬆️

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 3ee431d...d57c03b. Read the comment docs.

@patrickvonplaten patrickvonplaten changed the title [WIP] add t5 docs basis Add T5 to docs Mar 27, 2020
@patrickvonplaten patrickvonplaten mentioned this pull request Mar 27, 2020
4 tasks
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@thomwolf @LysandreJik - Could you maybe check the Tips?

Copy link
Member

Choose a reason for hiding this comment

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

I like the tips!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added encoder_outputs as input argument @sshleifer

Copy link
Contributor Author

Choose a reason for hiding this comment

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

In general I changed the docstring to "Bert"-style docstring since Bert seems to have the cleanest docstrings. @thomwolf

Copy link
Member

@LysandreJik LysandreJik left a comment

Choose a reason for hiding this comment

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

Love it!

Copy link
Member

Choose a reason for hiding this comment

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

I like the tips!

@LysandreJik LysandreJik merged commit fa9af24 into huggingface:master Mar 27, 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.

3 participants