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

TextStatsPlots.show_stats() should call TextStatsPlots.return_stats() #132

Open
meghdadFar opened this issue Mar 27, 2024 · 0 comments
Open

Comments

@meghdadFar
Copy link
Owner

Improvement:TextStatsPlots.show_stats() should call TextStatsPlots.return_stats()

Description:

We have created a new function TextStatsPlots.return_stats() for Datachat. It does most of what is done inside TextStatsPlots.show_stats() but instead of printing the stats in a tabulate table, it returns a json. The goal of this issue is to update TextStatsPlots.show_stats() so that it calls TextStatsPlots.return_stats() and then properly format the return value in a list so that it can be passed to tabulate. This will improve the consistency and allows us to get the stats only in one place i.e. TextStatsPlots.return_stats().

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

No branches or pull requests

1 participant