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

Show progress bar for the GToJSONConverter #16

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

pratheekshasn
Copy link
Collaborator

The converter looks quite unresponsive, especially while converting large projects. Adding a progress bar to fix this issue.

image

Implementation

Changes are made in GenerateHTML.vi to show the progress bar and the path of the file being converted. The front panel has been resized to show these, and is opened up as a dialog window when the converter is run.
image

@rajneesh
Copy link

How does one cancel the conversion? Previously, I believe they could hit the 'Abort Run' button.

@pratheekshasn
Copy link
Collaborator Author

pratheekshasn commented Jul 16, 2020

How does one cancel the conversion? Previously, I believe they could hit the 'Abort Run' button.

That hadn't occurred to me. Would it be all right if I changed the subVI to be abortable like so?
image
But this would also mean the front panel of that subVI doesn't close after I abort it... maybe I should look for a different way to do this.

@rajneesh
Copy link

The title bar is distracting. You could probably make it like a one button dialog which aborts the vi when the button is clicked. You'll have to don your UX hat and figure out the workflows one could go through.

@pratheekshasn
Copy link
Collaborator Author

pratheekshasn commented Sep 22, 2020

ProgressBar.vi
image

InitProgresssBar.vi
image

UpdateProgressBar.vi
image

CloseProgressBar.vi
image

GetAbort.vi
image

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