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

Using JSON API right after UDP Realtime protocol doesn't work #3589

Closed
1 task done
kryptan opened this issue Dec 14, 2023 · 3 comments
Closed
1 task done

Using JSON API right after UDP Realtime protocol doesn't work #3589

kryptan opened this issue Dec 14, 2023 · 3 comments
Labels
bug fixed in source This issue is unsolved in the latest release but fixed in master waiting for feedback addition information needed to better understand the issue

Comments

@kryptan
Copy link

kryptan commented Dec 14, 2023

What happened?

If you call JSON API right after using UDP realtime protocol then led strip gets stuck on the color set by the last UDP packet. Neither does it return to normal mode after a timeout nor does the JSON API call works. Subsequent JSON API calls do work though.

To Reproduce Bug

  1. Send UDP realtime packet to WLED. e.g. DNRGB.
  2. Wait 100 ms.
  3. Call json/state with "on": false.

Example Rust code: https://github.com/kryptan/wled-bug/blob/master/src/main.rs

Expected Behavior

JSON call should have an immediate effect or returning to normal mode after a timeout should work.

Install Method

Binary from WLED.me

What version of WLED?

WLED 0.14.0 (build 2310130)

Which microcontroller/board are you seeing the problem on?

ESP8266, ESP32

Relevant log/trace output

No response

Anything else?

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@kryptan kryptan added the bug label Dec 14, 2023
@blazoncek
Copy link
Collaborator

Try adding

  } else {
    strip.trigger();

or

  } else {
    strip.show();

between lines 197 and 198 in udp.cpp from main branch, compile and report back.
After this line:

strip.getMainSegment().freeze = false;

@blazoncek blazoncek added the waiting for feedback addition information needed to better understand the issue label Jan 2, 2024
@blazoncek
Copy link
Collaborator

Please provide feedback.

softhack007 pushed a commit to MoonModules/WLED that referenced this issue Feb 12, 2024
@blazoncek blazoncek added the fixed in source This issue is unsolved in the latest release but fixed in master label Feb 21, 2024
@blazoncek
Copy link
Collaborator

No feedback provided and most likely fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fixed in source This issue is unsolved in the latest release but fixed in master waiting for feedback addition information needed to better understand the issue
Projects
None yet
Development

No branches or pull requests

2 participants