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

Reduce number of seeds in vignette #117

Closed
ethanwhite opened this issue Mar 15, 2019 · 0 comments
Closed

Reduce number of seeds in vignette #117

ethanwhite opened this issue Mar 15, 2019 · 0 comments

Comments

@ethanwhite
Copy link
Member

@ethanwhite ethanwhite commented Mar 15, 2019

This call in the vignette takes a little over 5 minutes to run:

lda_model_set <- LDA_set(document_term_table = rodents$document_term_table,
                         topics = c(2:6),
                         nseeds = 100,
                        control = LDA_controls_list(quiet = TRUE))

The run time is linear with nseeds so we can cut that to 30 seconds using nseeds=5 which produces pretty similar results. This seems like a good trade to me to help folks exploring the package get off to a good start, but if there's a reason to keep the 100 that's OK too.

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

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.