-
Notifications
You must be signed in to change notification settings - Fork 31.9k
Update modeling doc strings FE -> IP #21106
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
Update modeling doc strings FE -> IP #21106
Conversation
|
The documentation is not available anymore as the PR was closed or merged. |
bca972c to
ea28fb3
Compare
sgugger
left a comment
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.
Note that add_code_sample_docstrings now ignores the processor_class argument, so you can just remove it entirely in all those model files. Left pointers on the first ones and then got tired of saying "Same here" ;-)
src/transformers/models/data2vec/modeling_tf_data2vec_vision.py
Outdated
Show resolved
Hide resolved
sgugger
left a comment
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.
Thanks for all the work on this!
NielsRogge
left a comment
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.
Thanks a lot!
What does this PR do?
Replaces references of feature extractors with image processors that had been missed in the first batch of changes
Before submitting
Pull Request section?
to it if that's the case.
documentation guidelines, and
here are tips on formatting docstrings.