-
Notifications
You must be signed in to change notification settings - Fork 26.9k
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
[docs] General doc fixes #28087
[docs] General doc fixes #28087
Conversation
Hi @amyeroberts! The CI error is:
Would this be fixed by removing
|
@stevhliu Yes, however this would stop NerPipeline being importable from the top level of transformers, which would be a breaking change. Instead, we can make an exception for it in the doc checks by adding it to DEPRECATED_OBJECTS (which it effectively is if TokenClassificationPipeline supercedes it) |
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
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 tidying all of this up!
* doc fix friday * deprecated objects * update not_doctested * update toctree
Cleans up a few things:
NerPipeline
from internal discussion here (basically it's identical to theTokenClassification
pipeline and it may cause confusion)perf_train_tpu.md
because it is empty and doesn't add any value to the docs