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

Documentation error on kno.wled.ge UDP Sync page #4496

Closed
Christanoid opened this issue Jan 16, 2025 · 2 comments
Closed

Documentation error on kno.wled.ge UDP Sync page #4496

Christanoid opened this issue Jan 16, 2025 · 2 comments

Comments

@Christanoid
Copy link

According to the Documentation there is no DNRGBW
Image

Looking at the code dnrgbw is in fact implemented, just not documented, as visible in udp.cpp

} else if (udpIn[0] == 4 && packetSize > 7) //dnrgb
{
  unsigned id = ((udpIn[3] << 0) & 0xFF) + ((udpIn[2] << 8) & 0xFF00);
  for (size_t i = 4; i < packetSize -2 && id < totalLen; i += 3, id++)
  {
    setRealtimePixel(id, udpIn[i], udpIn[i+1], udpIn[i+2], 0);
  }
} else if (udpIn[0] == 5 && packetSize > 8) //dnrgbw
{
  unsigned id = ((udpIn[3] << 0) & 0xFF) + ((udpIn[2] << 8) & 0xFF00);
  for (size_t i = 4; i < packetSize -2 && id < totalLen; i += 4, id++)
  {
    setRealtimePixel(id, udpIn[i], udpIn[i+1], udpIn[i+2], udpIn[i+3]);
  }
}

Sadly I don't know where / if the documentation source code is set and can't find it, so this fyi instead. Sorry.

@dosipod
Copy link
Contributor

dosipod commented Jan 17, 2025

@Christanoid
Copy link
Author

Thanks, will adapt it there after testing if it acutally works.

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

2 participants