search

ADVRHumanoids / XBotGUI

XBot Graphical User Interface for XBot powered robots
3 stars 1 forks source link

GUI closing not working #42

Closed pietrobalatti closed 6 years ago

pietrobalatti commented 6 years ago

CTRL+C is always intercepted by the GUI (red message in the terminal) but then it gets almost always stuck there and I need to close it sending "killall XBotGUI -9" from another terminal.

Could it be an idea to send this killall command when this keys combination is intercepted? I don't know if it is even possible or if it is a too "dirty" solution, it's just my two cents.

alessandrosettimi commented 6 years ago

This is not happening on my machine, does this happen to you always? Simple test: run it, do nothing, then CTRL+C.

pietrobalatti commented 6 years ago

No not always, if I close it at the beginning it works (probably there are not a lot of things going on). If I am in the middle of the task it gets stuck..

alessandrosettimi commented 6 years ago

The next time this issue appear, please write here what exactly you were doing. I think it is some thread waiting some something that freeze the kill.

The solution of putting "killall" inside the gui is super-dirty :laughing:

pietrobalatti commented 6 years ago

I think we solved it, so basically the GUI was running on all the available cores and this was causing problems. It was enough to make it run just on one core by setting:

taskset -c 0 XBotGUI

and now it is much lighter!

alessandrosettimi commented 6 years ago

A greedy GUI :)

pietrobalatti commented 6 years ago

Super greedy ?

Il giorno 31 gen 2018, alle ore 18:51, Alessandro Settimi notifications@github.com<mailto:notifications@github.com> ha scritto:

A greedy GUI :)

- You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHubhttps://github.com/ADVRHumanoids/XBotGUI/issues/42#issuecomment-362014609, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AJvrhxf4R2XiuILIhrCquztdMO-VE_ftks5tQKgNgaJpZM4RuE84.