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

oscilloscope continues to run after stop #2563

Closed
walterbender opened this issue Sep 13, 2020 · 7 comments
Closed

oscilloscope continues to run after stop #2563

walterbender opened this issue Sep 13, 2020 · 7 comments
Assignees
Labels

Comments

@walterbender
Copy link
Member

Checking the console output, line 103 in oscilloscope.js continues to generate spam after the stop button has been pressed.

@sksum
Copy link
Member

sksum commented Sep 25, 2020

"oscilloscope running" should log every frame while the widget is open , does it still log when the widget is closed ?

@walterbender
Copy link
Member Author

I think it should not be running when the program is not running, even if it is open. Is that possible?

@walterbender
Copy link
Member Author

(Check the console output to see what I mean... when the oscilloscope is running, the console is bombarded with output even if there is nothing else running.)

@ricknjacky
Copy link
Member

Changes to the code :-
Screenshot (369)

It should not be running when the program is not running, even if it is open.

The console output that I got:-
Screenshot (370)

Does this match the behavior you expect? @walterbender

@walterbender
Copy link
Member Author

That suppresses the log output but the code presumably is still running. We need to stop the code from running.

@ricknjacky
Copy link
Member

When the widget is open the oscilloscope code should run? right? When we close the widget window it stops ig.

@walterbender
Copy link
Member Author

It should also stop when the program execution stops, regardless of whether or not the widget is open.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants