- 
                Notifications
    You must be signed in to change notification settings 
- Fork 31k
Add T5 to docs #3461
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
Add T5 to docs #3461
Conversation
efa2444    to
    c808ded      
    Compare
  
    | Codecov Report
 @@           Coverage Diff           @@
##           master    #3461   +/-   ##
=======================================
  Coverage   77.79%   77.80%           
=======================================
  Files         100      100           
  Lines       17049    17051    +2     
=======================================
+ Hits        13264    13267    +3     
+ Misses       3785     3784    -1     
 Continue to review full report at Codecov. 
 | 
        
          
                docs/source/model_doc/t5.rst
              
                Outdated
          
        
      There was a problem hiding this comment.
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?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the tips!
        
          
                src/transformers/modeling_bart.py
              
                Outdated
          
        
      There was a problem hiding this comment.
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
        
          
                src/transformers/modeling_t5.py
              
                Outdated
          
        
      There was a problem hiding this comment.
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
f231dee    to
    0de7b45      
    Compare
  
    There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love it!
        
          
                docs/source/model_doc/t5.rst
              
                Outdated
          
        
      There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the tips!
modeling_t5.pyandmodeling_tf_t5.py