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

Unclear running status. #56

Closed
Shawn-Guo-CN opened this issue Sep 30, 2019 · 4 comments
Closed

Unclear running status. #56

Shawn-Guo-CN opened this issue Sep 30, 2019 · 4 comments

Comments

@Shawn-Guo-CN
Copy link
Contributor

Shawn-Guo-CN commented Sep 30, 2019

Currently (version 0.5.2), it's hard to track the running status of pipeline models, as all the related information won't appear in the GUI. The inconvenience found yet:

  1. Hard to tell whether a pipeline is done, even with the output from background console.

  2. When a loaded pipeline is incompatible with the running Spikely, users have to refer to the background console to find out what is missing.

My suggestion for updating would be, use QTextEdit to display text from stdout as well as stderr.

@Shawn-Guo-CN Shawn-Guo-CN changed the title Unclear whether running pipeling is done. Unclear running status. Oct 1, 2019
@rogerhurwitz
Copy link
Contributor

I added a messaging queue shared by parent and child processes to allow the process running the pipeline (child) to signal the app (parent) upon completion of run. I hooked that queue up to a message box that serves as an indicator. I have no idea of whether it will work on Windows. :)

@Shawn-Guo-CN
Copy link
Contributor Author

I added a messaging queue shared by parent and child processes to allow the process running the pipeline (child) to signal the app (parent) upon completion of run. I hooked that queue up to a message box that serves as an indicator. I have no idea of whether it will work on Windows. :)

The message box could work on Windows: it pops out after "run" button is clicked, and it could be closed manutally or automatically. This is very helpful. But, based on the current tests, I realised another issue #61 .

So, I close this issue but open a new one.

@Shawn-Guo-CN
Copy link
Contributor Author

Shawn-Guo-CN commented Oct 2, 2019

Just found a new problem. If the pipeline crashes in background, there won't be any signal from the child processes to the parent process. Thus: i) users may manually close the message box window; ii) the message box will be there forever. In neither case, the users could know that the pipeline crashes in background.

This could be solved if we just print the stdout from that pipeline, as mentioned in #61 .

@colehurwitz
Copy link
Member

colehurwitz commented Oct 2, 2019

Moving to new issue #67

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

3 participants