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

Improve the progress bars when profiling datasets #7

Open
neomatrix369 opened this issue Sep 20, 2020 · 2 comments
Open

Improve the progress bars when profiling datasets #7

neomatrix369 opened this issue Sep 20, 2020 · 2 comments
Labels
3. low-priority Non-urgent issue, can be fixed at a later stage enhancement New feature or request good first issue Good for newcomers hacktoberfest Classify topic. Part of the Hacktoberfest 2020 (https://hacktoberfest.digitalocean.com) help wanted Extra attention is needed UI/UX feedback

Comments

@neomatrix369
Copy link
Owner

neomatrix369 commented Sep 20, 2020

We have progress bars shown for each level of progress while datasets are profile, except that now we see a lot of them.

image

Reducing the details and just having a few that show all the stats would be helpful from UX/UI point of view.

Related to #3

Suggestions to potential solution

  • (option 1: ideal solution): show all 3 levels of progress bars and only 3 progress bars which dynamically change and show progress
    • currently, the 2nd and 3rd level progress bars are getting re-created (and appended to the visual cue), we would like this to not happen
  • (option 2: easy) disable the 3 level (row-level) progress bar
    • hide this behind a toggle switch that can be passed as a param, by default the parameter switches off the progress bar
  • (option 3: half-way): show all the details in two levels of progress bar which change dynamically and do not get re-created for each iteration

How do we know it works well and there is no regress?

  • run the notebooks in the notebooks folder to see if the changes are working as expected
  • run the test coverage shell script to see tests pass 100% and coverage is 100%
@neomatrix369 neomatrix369 added enhancement New feature or request UI/UX feedback labels Sep 20, 2020
@neomatrix369 neomatrix369 added this to To do in NLP Profiler via automation Sep 20, 2020
@neomatrix369 neomatrix369 added help wanted Extra attention is needed Hacktoberfest labels Sep 20, 2020
@neomatrix369 neomatrix369 moved this from To do to Backlog in NLP Profiler Sep 27, 2020
@strivedi02
Copy link

I'll try to put one progress bar instead of multiple progress bars.

@neomatrix369
Copy link
Owner Author

neomatrix369 commented Sep 30, 2020

I'll try to put one progress bar instead of multiple progress bars.

Will the single progress bar convey most if not all the feedback the current progress bars convey?

Please also read the updated description, so you get some ideas as to how to go about doing it and also alternative solutions.

@neomatrix369 neomatrix369 added hacktoberfest-accepted Approved/merged. Part of the Hacktoberfest 2020 (https://hacktoberfest.digitalocean.com) 3. low-priority Non-urgent issue, can be fixed at a later stage hacktoberfest Classify topic. Part of the Hacktoberfest 2020 (https://hacktoberfest.digitalocean.com) good first issue Good for newcomers and removed Hacktoberfest2019 hacktoberfest-accepted Approved/merged. Part of the Hacktoberfest 2020 (https://hacktoberfest.digitalocean.com) labels Oct 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. low-priority Non-urgent issue, can be fixed at a later stage enhancement New feature or request good first issue Good for newcomers hacktoberfest Classify topic. Part of the Hacktoberfest 2020 (https://hacktoberfest.digitalocean.com) help wanted Extra attention is needed UI/UX feedback
Projects
NLP Profiler
  
Backlog
Development

No branches or pull requests

2 participants