Skip to content

fix issue with report's show arguments#11

Merged
maximtrp merged 1 commit intomaximtrp:mainfrom
Nfsaavedra:fix_show
Aug 14, 2024
Merged

fix issue with report's show arguments#11
maximtrp merged 1 commit intomaximtrp:mainfrom
Nfsaavedra:fix_show

Conversation

@Nfsaavedra
Copy link
Copy Markdown
Contributor

tmp.report(model, docs=filtered_bcts, height=400, width=250, show_topics=False)

When using the keyword show_topics, show_words or show_docs, the program would raise the following error when trying to change the topic in the visualization:

NameError: free variable 'topics_plot_output' referenced before assignment in enclosing scope

This PR fixes that error.

@sonarqubecloud
Copy link
Copy Markdown

@maximtrp maximtrp merged commit 24e48c5 into maximtrp:main Aug 14, 2024
@maximtrp
Copy link
Copy Markdown
Owner

Thank you!

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