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

[BUG] CPU activity when the app is paused syncing #142

Closed
vlad-cotfas opened this issue May 12, 2020 · 13 comments · Fixed by #148
Closed

[BUG] CPU activity when the app is paused syncing #142

vlad-cotfas opened this issue May 12, 2020 · 13 comments · Fixed by #148
Labels
bug Something isn't working gui macOS

Comments

@vlad-cotfas
Copy link

Catalina 10.15.4
Maestral 1.0.2

cpu

cpuActivity.mov.zip

See the attached files, I see it is still doing CPU activity when the Maestral app is paused syncing.

@vlad-cotfas
Copy link
Author

Also when the app is still in pause, it has energy impact on the battery..

image

I normally put the Maestral app on pause when I`m on battery or I need the full bandwidth of the internet and CPU/RAM resources for intensive work.

@samschott
Copy link
Owner

Yes, I am aware of the issue and already have a fix for the next release...

@samschott
Copy link
Owner

It's the GUI and not the daemon process which is causing the high CPU usage. Therefore, you can mitigate this by running the daemon without the GUI through the command line interface for now.

It's not as convenient, unfortunately, but you can start a GUI as necessary and it will attach itself to the running daemon. You'll notice that the "Quit Maestral" menu entry changes to "Quit Maestral GUI" which indicates that quitting the GUI will leave the daemon running.

@vlad-cotfas
Copy link
Author

Yes, but I kinda need the statusbar to know what is going on.. :(

@vlad-cotfas
Copy link
Author

If I`m switching to the terminal version from the GUI, I can switch it back with cli:

maestral gui

And to pause/start:

maestral pause|resume

And to see what has been synced:

maestral status

Do I have to do the:

maestral autostart

To register it with Mac startup?

Right?

@samschott
Copy link
Owner

Yes! Although maestral autostart only registers the daemon to start on login and not the GUI.

I'll see if I can get a beta release out with bug fixes soon.

@vlad-cotfas
Copy link
Author

From the GUI I have disabled the autostartup, and from the CLI when I do:

maestral autostart

I get:

Autostart is disabled.

I did not restarted my mac yet to see if is ok.

@vlad-cotfas
Copy link
Author

It does not restart the daemon on mac restart.

cotfas:~ work$ maestral status
Maestral daemon is not running.

And after restart, I got again the GUI maestral dialog error from #137

@samschott
Copy link
Owner

The GUI command to enable autostart is:

maestral autostart -Y

maestral autostart only returns the current status. Yeah, I am working on #137 but its difficult to debug...

@vlad-cotfas
Copy link
Author

cotfas:~ work$ maestral autostart -Y
Enabled start on login.
cotfas:~ work$ maestral autostart
Autostart is enabled.
cotfas:~ work$

Perfect :)

Maybe update the repo readme file to be updated with the commands for mac as well, I did not knew about -Y param.

And the help guide does not give to much info:

cotfas:~ work$ maestral -h
usage: main [-h] [-c CONFIG_NAME] [--cli] [--frozen-daemon]

optional arguments:
-h, --help show this help message and exit
-c CONFIG_NAME, --config-name CONFIG_NAME
Configuration name
--cli Forward calls to CLI.
--frozen-daemon Start daemon only
cotfas:~ work$

@vlad-cotfas
Copy link
Author

Will let you know on next reboot if the maestral autostart -Y worked. Thanks.

@vlad-cotfas
Copy link
Author

vlad-cotfas commented May 13, 2020

Wow, definitely will use the daemon/cli instead of GUI from now on as I see the memory/cpu/energy impact looks way better.

@samschott good job, keep up the good work, don`t forget about #144 please. :)

@samschott
Copy link
Owner

There is a beta out now which reduces the CPU usage of the GUI to more reasonable levels.

Wow, definitely will use the daemon/cli instead of GUI from now on as I see the memory/cpu/energy impact looks way better.

Yes, the cocoa UI uses about as much memory as the daemon itself...

@samschott samschott added bug Something isn't working gui macOS labels May 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working gui macOS
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants