-
Notifications
You must be signed in to change notification settings - Fork 28.2k
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
Fix OPT doc tests #18365
Fix OPT doc tests #18365
Conversation
The documentation is not available anymore as the PR was closed or merged. |
Hi @ArthurZucker Do you know why the expected value has extra ``` ?
|
Actually it seems that the automatic doc préparation that adds a line does not affect the TFOPTForCausalLM forward doc. Thus I removed the manually created example and defeated to using add_code_sample which worked in that case. |
Thank you. I will check this on Monday. |
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.
Thank you for the fix, @ArthurZucker ! LGTM, especially all tests pass now.
Let's wait an approval from some core maintainers.
Regarding the mystery , it is due to this line
|
I forgot to change the name in the |
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 fixing!
What does this PR do?
Fixes the doctest for OPT by adding slight modifications.