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

Add support for Skydimo USB lights #934

Open
flobere opened this issue Sep 7, 2024 · 5 comments
Open

Add support for Skydimo USB lights #934

flobere opened this issue Sep 7, 2024 · 5 comments

Comments

@flobere
Copy link

flobere commented Sep 7, 2024

Feature request

Add support for Skydimo USB lights

What problem does this feature solve?

compatibility with Skydimo lights- simple plug and play USB devices which can be had cheaply from Aliexpress. They are apparently similar to the Adalight implementation.

What does the proposed API look like?

Skydimo device support in LED controller config

How should this be implemented in your opinion?

I have no coding knowledge, but a feature request outlining details on the hardware and possible implementation was raised on the hyperion.ng project here, and later added (i'm unaware if linking to this is helpful or good practice, i'm new to using github)

Are you willing to work on this yourself?

No, but will happily donate to the project

@jontstaz
Copy link

Out of interest, have you tried using them just by using the Adalight config? I'm considering buying a few sets of these for my monitors but I run Arch Linux and the Skydimo software only supports Windows.

@flobere
Copy link
Author

flobere commented Oct 30, 2024

No luck with the adalight config. It's unfortunate because they're a great AIO product. hyperion.ng works out of the box (effects, colour etc), but has no support for wayland/ pipewire screen capture and no intention of adding it in the foreseeable future. This project neatly supports it, but no support for the skidimo lights yet. I'm hoping that the latter is a much simpler implementation.

@flobere flobere closed this as completed Oct 30, 2024
@flobere flobere reopened this Oct 30, 2024
@awawa-dev
Copy link
Owner

If you can provide documentation describing the protocol, then I think there would be a good chance to implement it. But I won't blindly copy and paste code from Hyperion. No official protocol documentation => the product is not as good as one might think and unless it's super special, there's no point in risking reverse engineering.

@flobere
Copy link
Author

flobere commented Oct 30, 2024

thanks for the reply. I only praise the product from a layperson/consumer standpoint- it's affordable, cleanly packaged and plug-and-play. Like any of the fine blackboxed wonders on aliexpress or temu, I doubt technical data is forthcoming unless reverse engineered. the linked comment made it sound like more of a matter of adding a tweaked adalight entry than reinventing the wheel, though.

said commentIt is basically Adalight but Bytes 1 2 and 3 are "Ada", Bytes 4 and 5 are blank (0x00), Byte 6 is Led Count

Exemple for 65 leds:

Skydimo: Ada..A.......................................................... Hex: 41 64 61 00 00 41 00 00 00 ...

Hyperion: Ada.@........................................................... Hex: 41 64 61 00 40 15 00 00 00 ...

@awawa-dev
Copy link
Owner

a loose note related to the topic looking at this hack: the participation of Chinese engineers in modifying the Adalight protocol consisted in completely removing the only mechanism that provided any kind of data consistency check (CRC for header), i.e. they basically broke it completely. However, I don't have these LEDs, so it would be hard to implement this example of bad engineering blindly. If someone is willing and will implement and test it, I have nothing against it.

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

No branches or pull requests

3 participants