Skip to content

Docstring and module updates #150

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Jun 15, 2022
Merged

Docstring and module updates #150

merged 4 commits into from
Jun 15, 2022

Conversation

rwedge
Copy link
Contributor

@rwedge rwedge commented Jun 15, 2022

  • Makes Elmo and UniversalSentenceEncoder accessible in the nlp_primitives.tensorflow space. It's easy for the docs to then point at that submodule and get all the tensorflow primitives.
  • Updates the syntax of some docstrings that were causing warnings when building the docs

@rwedge rwedge requested review from gsheni June 15, 2022 22:37
@rwedge rwedge merged commit a1a71e4 into main Jun 15, 2022
@rwedge rwedge deleted the docstring-and-module-updates branch June 15, 2022 22:57
@rwedge rwedge mentioned this pull request Jun 16, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gsheni gsheni Awaiting requested review from gsheni

Assignees

@rwedge rwedge

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rwedge @gsheni