Skip to content
This repository has been archived by the owner on Jun 8, 2022. It is now read-only.

Ambient.py #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Ambient.py #1

wants to merge 3 commits into from

Conversation

aretaon
Copy link

@aretaon aretaon commented Jan 4, 2021

Hi,

running OpenRGB under Ubuntu I found the missing screen capture support in Ambient.py quite annoying (since the remaining scripts perform well under Linux) and patched together a workaround.

319d633 uses X11 to get the PIL Image from the current screen, but for performance reasons it requires compiling some C code on the machine beforehand (all credit goes to https://stackoverflow.com/questions/69645/take-a-screenshot-via-a-python-script-on-linux). Having the compiled program, Ambient.py runs very well on my machine.

I also played around using k-means clustering instead of just the average RGB value to determine the LED-setting (fd8ea3a). This is somewhat experimental as it required smoothing out the colors over multiple frames (in this case 3) to prevent flickering due to sampling differences. It works well for me for gaming but it requires scikit-learn for clustering and is not tested on Win or Mac.

Hope you find something oft his useful.
Cheers

@CoffeeIsLife87
Copy link
Owner

CoffeeIsLife87 commented Jan 5, 2021

This looks great so far :)

I am gonna throw in some try/except for imports and then it should be good to go

Ambient has the same issues on my windows machine but I guess the point of this MR is linux support so I am gonna merge it anyways

@CoffeeIsLife87
Copy link
Owner

Ambient.zip

Make any adjustments you want to that and let me know when you think it is ready

Then I can test and merge if it is working as it should

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants