BradyBrenot / huestacean

Philips Hue control app for desktop with screen syncing. C++ with Qt Quick GUI.
http://huestacean.com
Apache License 2.0
566 stars 54 forks source link

Investigate perf, especially on older hardware #31

Closed BradyBrenot closed 6 years ago

BradyBrenot commented 6 years ago

Laptop - 2007 Macbook Pro

Windows

Mac OS

BradyBrenot commented 6 years ago

So far:

On Windows/DX11

Not sure if I can do anything about DXGI screen cap performance, but one thing I can do is reduce the size of the texture we pull off the GPU, since we need to drop pixels to even use it on the CPU in a reasonable time anyways. So:

Reduce size of image before pulling off GPU, not after #36

klodeckel1990 commented 6 years ago

Just tested it on the MacBook Pro 2015 with 2.7GHz Intel Core i5 and Intel Iris 7100 GPU, and there are significant performance problems. It is not possible to play a YouTube video and the whole system is jerking, also the Entertainment-Group is not synching very well.

BradyBrenot commented 6 years ago

macOS runs pretty poop generally. It may be the way screen_capture_light is grabbing the screen.

Developing on macOS is a bit of a pain for me, at the moment, but I'll have to just get over it.

BradyBrenot commented 6 years ago

Shoving this off. Maybe I'll look into it on the weekend.

BradyBrenot commented 6 years ago

Work for macOS in #62

Windows appears fine now that I'm resizing images before returning them from the GPU, so nothing to do there.