Skip to content
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 link to Hoffman paper (online VB LDA) #3133

Merged
merged 5 commits into from
May 12, 2021

Conversation

jonaschn
Copy link
Contributor

@jonaschn jonaschn commented May 5, 2021

The previous link was Matthew Hoffman's Google Scholar profile or not the official one.
Use full author names in the first occurrence and first author only afterward.
Furthermore, use latex symbols for original parameter names from Hoffman et al.

The previous link was Matthew Hoffman's Google Scholar profile or not the official one.
Use full author names in the first occurrence and first author only afterward.
Copy link
Collaborator

@mpenkov mpenkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @jonaschn for picking up on this.

Is it possible to include the hyperlink only once, e.g. in the module docstring, and then reference it from within the function docstrings? I think RST allows something like this.

https://www.sphinx-doc.org/en/master/usage/restructuredtext/basics.html#hyperlinks

I'd rather avoid having the same long hyperlink (or hyperlinks, if we're showing more than one) appear in multiple places.

and use latex symbols for original parameter names
@mpenkov
Copy link
Collaborator

mpenkov commented May 12, 2021

Thank you for your contribution @jonaschn !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants