-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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 of documentation after package reorganization #349
Conversation
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
This pull request introduces 1 alert when merging 09d4678 into 8d7bef6 - view on LGTM.com new alerts:
|
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.
import tokenizers as follows:
import nemo.collections.nlp as nemo_nlp
nemo_nlp.data.SentencePieceTokenizer
same for modules:
import nemo.collections.nlp as nemo_nlp
nemo_nlp.nm.[classifier]
pytorch_utils.py is gone after refactored
Signed-off-by: Yang Zhang <yangzhang@nvidia.com>
This pull request introduces 1 alert when merging 8bf4389 into 8d7bef6 - view on LGTM.com new alerts:
|
This pull request introduces 1 alert when merging 9b5acd4 into 8d7bef6 - view on LGTM.com new alerts:
|
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
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 :)
Fixed several Sphinx-related issues that probably resulted from the reorganizations of a) the packages in general and b) of NLP collection in particular.
DONE:
For example, .. automodule:: nemo.collections.nlp.huggingface.bert ->
.. automodule:: nemo.collections.nlp.nm.trainables.common.huggingface.bert