Xanthos-Code / vintel

EVE Online Visual Intel Tool
112 stars 59 forks source link

High CPU usage #109

Open emptyway opened 8 years ago

emptyway commented 8 years ago

f7e71fbfe8c54dd184be4f6c4afdf4de

im multi boxing with 3 Eve account.. and notice high CPU usage on Vintel... is this normal?

3vi1 commented 8 years ago

Seems exceptionally high. I haven't really looked at performance myself, but Xanthos may already be aware of things that are yet to be optimized.

I'll do a profiling pass myself tonight and see what I can turn up.

Xanthos-Code commented 8 years ago

That looks normal to me - it should clam down to around 10% then spike up to ~20. Its a busy app and does things that are not great for overall system performance like scanning logs and updating the map every couple of seconds.

On Wed, Jul 6, 2016 at 11:31 AM, Simon notifications@github.com wrote:

[image: f7e71fbfe8c54dd184be4f6c4afdf4de] https://cloud.githubusercontent.com/assets/1506890/16629951/1fe8aeba-4386-11e6-8a1f-d2c885f70667.png

im multi boxing with 3 Eve account.. and notice high CPU usage on Vintel... is this normal?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Xanthos-Eve/vintel/issues/109, or mute the thread https://github.com/notifications/unsubscribe/APr2Bpo-a7tgl5MWSckQU7hrDNfi0ybvks5qS_RqgaJpZM4JGXtj .

Xanthos-Code commented 8 years ago

Its a trade off - we can make it impact the system less but then KOS checks will be slower and the map redraw will happen less frequently. There may be some thing we can do. Also, make sure you don't have years worth of logs for it to scan. That will impact performance more than anything.

On Wed, Jul 6, 2016 at 3:41 PM, Xanthos Eve xanthos.eve@gmail.com wrote:

That looks normal to me - it should clam down to around 10% then spike up to ~20. Its a busy app and does things that are not great for overall system performance like scanning logs and updating the map every couple of seconds.

On Wed, Jul 6, 2016 at 11:31 AM, Simon notifications@github.com wrote:

[image: f7e71fbfe8c54dd184be4f6c4afdf4de] https://cloud.githubusercontent.com/assets/1506890/16629951/1fe8aeba-4386-11e6-8a1f-d2c885f70667.png

im multi boxing with 3 Eve account.. and notice high CPU usage on Vintel... is this normal?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Xanthos-Eve/vintel/issues/109, or mute the thread https://github.com/notifications/unsubscribe/APr2Bpo-a7tgl5MWSckQU7hrDNfi0ybvks5qS_RqgaJpZM4JGXtj .

3vi1 commented 8 years ago

That still seems super inefficient, for the amount of work being done. I didn't get a chance to look at it last night, but let me have a look tonight/this weekend.

3vi1 commented 8 years ago

Went ahead profiled it in the background, and am more in agreement with Xanthos now.

The develop branch with Qt5 runs at about 5% of one CPU on my system. The vast majority of that time is spent in the underlying Qt and Python libs. The biggest hit in the app is updating the map, and that only accounts for about 3.3% of the cumulative time (and most of that's used in the Beautiful Soup libraries).

I don't see any major bottlenecks in the app itself, but I'll keep running with the profiler so that I can catch it if I notice it in normal usage this weekend.

emptyway commented 8 years ago

my current work around is to use only 1 region map setting. it reduced the workload a lot

emptyway commented 8 years ago

btw... i didnt have such high loads on previews versions( earlier than 1.2.3)

Xanthos-Code commented 8 years ago

Interesting, did / can you spend any time with 1.2.2. That would help us narrow it down to a handful of commits. In 1.2.3 significant changes were made to chat parser and image scaling in intel panel. Under 1.2.3 if you quit the eve client does it get better after a bit of settling? If so that would steer us toward the chat parser. In 1.2.2 we added main window event filtering for mouse events and some other parser work so those would also both be suspect.

Sounds like we should consider some kind of automation to performance test before releases also.

emptyway commented 8 years ago

Single EvE Client: I notice high cpu usage when zoom in and scrolling the map around.

Multiple EvE Client:

intel2

intel

Xanthos-Code commented 8 years ago

I have created a new ticket for the duplicate intel.