Can't Load profile if process already running

[heyeddi]

Because of this:

python-umonitor/umonitor/umonitor.py

Line 119 in 6175f3f

self._prevent_duplicate_running()

Can't load a profile when the process is already running, which is kind of annoying.

I had to delete my "laptop" profile (no external screens) because the thing kept turning off screens when I disconnected the HDMI cable, and I could not load it manually while I was debugging.

Is there a reason for this?

[rliou92]

If you manually load a profile, the existing running process will detect a screen change and change the screen according to your profile afterwards, overriding the manual load. Is this what you wanted to happen?

[heyeddi]

@rliou92 that is the thing, it happened a couple of times the running process did not detect the setup, I had to trigger it manually.

My solution was to kill it and start it again, but umonitor -l profile should force the given profile without needing to kill the process, IMO

Or maybe add a new option to reload the whole thing for such cases, for instance umonitor -r which would:

• figure out if the process is live and/or daemonized
• if process exist:
  • kill the process existing
  • start a new one, daemonized
• else: Do same thing as --autoload

What do you think?

[rliou92]

Yes, I like both ideas!

Though the root cause of the problem is that the running process failed to detect a change in setup. Are you able to reproduce such a scenario?

[heyeddi]

towards the root cause, I think it might be my hardware, I am currently using autorandr (don't remember why I switched), and I have some issues, I am going to remove it to give umonitor a try again.

I'll come back wen I see issues.

[rliou92]

Ok. Monitor setup detection is pretty tricky.

[heyeddi]

It is, that is why I am suggesting some workarounds in case sometimes it just doesn't work :)
Just now I was fighting with my monitors, I use a dock station from work that it is said to be troublesome .. so sometimes it just doesn't cooperate

[rliou92]

I implemented this if you still need it. Not sure how the program responds in practical situations, but the functionality is there.