search

dsdude123 / GroovyRP

Groove Music rich presence for Discord
MIT License
35 stars 5 forks source link

High CPU usage #22

Open Draconov opened 3 years ago

Draconov commented 3 years ago

image How is this consuming that much of cpu core

AliMirlou commented 3 years ago

I have the same issue in the latest version (1.0.0) where two processes of GroovyRP and GroovyRPCore are created when opening the program and they take about 13 and 10 percent of CPU respectively (only when Groove Music is playing something)! I waited some time, hoping they would flat but they kept using CPU.

image

I tried reconfiguring it to foreground and disabling auto-open and update check, but still the same:

image

AliMirlou commented 3 years ago

It also causes Windows Explorer and some parts of the system to crash after some time of working!

dsdude123 commented 3 years ago

@AliMirlou Please show system logs that prove this. There has been zero other reports of this nor has this been encountered in testing.

As for CPU usage, while there are some spots where improvements can be made that might help bring it down, this isn't really a major issue and won't be treated with priority.

AliMirlou commented 3 years ago

@dsdude123 For the crashes, Can you help me on finding the logs? Where should I look?

For the CPU usage, I don't think it's because of any inefficiencies because the preview versions don't have this issue. I think something has gone wrong like an infinite loop or such!

dsdude123 commented 3 years ago

@AliMirlou Regarding crashes, best guess would be to check Windows Event Viewer to see if you can find anything that ties the issues you are having to GroovyRP.

Regarding CPU usage, it's likely an inefficiency. Both current and past versions in a sense run in an infinite loop as it is always checking for updates to currently playing music and doesn't stop until the application is closed. What has changed though is the addition of UI elements (notification tray element) and the method of getting the media information changed. Most likely need to add a adjustable polling rate of the UI events and the media information.

RedcodesDev commented 3 years ago

image I also got the same issue. It uses about 10% of the CPU

Maxmgmt commented 7 months ago

img was working around 10% but goes up, do have a older i7 this much usages is the same as an instance of league of legends. Doesn't update to change song info too.