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 _bertopic.py #895

Closed
wants to merge 1 commit into from
Closed

Update _bertopic.py #895

wants to merge 1 commit into from

Conversation

nbalepur
Copy link

Adding functionality in topics_over_time to allow users to specify how many terms under each topic they want to see at each timestep t. Right now, the default value is 5, but there are use cases where the user may want to see more than 5

@MaartenGr
Copy link
Owner

Thank you for the PR. The difficulty with opening up the top_n_words_per_time variable here is that it then should be done for every other variation of BERTopic that generates top n words. This makes the combination with the original top_n_words a bit more complex and less intuitive especially when it needs to be implemented for many variations.

@MaartenGr MaartenGr closed this Feb 21, 2023
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.

None yet

2 participants