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

Scrolling text: <trail> value not available #688

Closed
Haifeif opened this issue Apr 27, 2024 · 9 comments
Closed

Scrolling text: <trail> value not available #688

Haifeif opened this issue Apr 27, 2024 · 9 comments
Assignees
Labels
enhancement New feature or request

Comments

@Haifeif
Copy link

Haifeif commented Apr 27, 2024

The effect of WLED is used to display meta data of songs played. So far, so good. But the there is a value at the WLED web gui, that is always set to 128 at startup. It produces a kind of shadow to the scrolling text which makes it harder to read. Using the web gui to set it to 0 is possible and solves the problem of the text shadow. But I can't find the value at the iob-adapter.

When I change it by the web-gui, the adapter shows it at <wled.0.083af2ad4c6c.seg.0.c1>. But is read only ...

If there is a parameter already, please let me know. If not, I would appreciate to have it at the adapter!

Keep up with the great work!

Rgds, Uwe

@HaggardFFM
Copy link
Contributor

@Haifeif
I changed the C1 - C3 Values to writeable.
If @DutchmanNL merge my pull request you will have the possibility to change this values.

regards
Harald

@Haifeif
Copy link
Author

Haifeif commented May 12, 2024

Thanx, Harald!
Danke, Harald!

Entschuldige meine Unkenntnis: Gibt es dann eine neue Version des Adapters? Oder kann ich irgendetwas "manuell" tun?

LG Uwe

@HaggardFFM
Copy link
Contributor

Hallo Uwe,
da @DutchmanNL meinen Pull request noch nicht eingefügt hat, gibt es noch keine neue Version.
Was du machen kannst (manuell) ist der Austausch oder die Bearbeitung der stateAttr.js Datei.
ich hänge dir die neue Version hier an.
stateAttr.zip

Diese Datei kopierst du dann in deinen IO Broker. unter node_modules/iobroker.wled/lib und ersetzt die dort vorhandene Datei
(Vorher eine Sicherungskopie anlegen) oder die originale einfach umbenennen.

Anschliessend die Instanz neustarten, danach sollten die Werte c1 - c3 beschreibar sein.

VG Harald

@DutchmanNL
Copy link
Contributor

Hallo Uwe,

da @DutchmanNL meinen Pull request noch nicht eingefügt hat, gibt es noch keine neue Version.

Was du machen kannst (manuell) ist der Austausch oder die Bearbeitung der stateAttr.js Datei.

ich hänge dir die neue Version hier an.

stateAttr.zip

Diese Datei kopierst du dann in deinen IO Broker. unter node_modules/iobroker.wled/lib und ersetzt die dort vorhandene Datei

(Vorher eine Sicherungskopie anlegen) oder die originale einfach umbenennen.

Anschliessend die Instanz neustarten, danach sollten die Werte c1 - c3 beschreibar sein.

VG Harald

Ich kümmere mich diese Woche drum, danke für den PR

@HaggardFFM
Copy link
Contributor

Hallo Uwe,
da @DutchmanNL meinen Pull request noch nicht eingefügt hat, gibt es noch keine neue Version.
Was du machen kannst (manuell) ist der Austausch oder die Bearbeitung der stateAttr.js Datei.
ich hänge dir die neue Version hier an.
stateAttr.zip
Diese Datei kopierst du dann in deinen IO Broker. unter node_modules/iobroker.wled/lib und ersetzt die dort vorhandene Datei
(Vorher eine Sicherungskopie anlegen) oder die originale einfach umbenennen.
Anschliessend die Instanz neustarten, danach sollten die Werte c1 - c3 beschreibar sein.
VG Harald

Ich kümmere mich diese Woche drum, danke für den PR

Danke schön.

DutchmanNL added a commit that referenced this issue May 13, 2024
@DutchmanNL
Copy link
Contributor

DutchmanNL commented May 13, 2024

merge done, auch einige updates mit dependencies gemacht bitte die neue beta testen :)

danke @HaggardFFM !

@HaggardFFM
Copy link
Contributor

@DutchmanNL looks good.
Acutally there are no errors in the log.

@HaggardFFM
Copy link
Contributor

@DutchmanNL
No errors after 24 h in the new Beta.
I think you can generate a new Release Version.
best regards
Harald

@DutchmanNL
Copy link
Contributor

@DutchmanNL No errors after 24 h in the new Beta. I think you can generate a new Release Version. best regards Harald

perfect thanks, will check if there are some other small stuff which are worth to include and make a new release + publish request to stable this weekend

DutchmanNL pushed a commit that referenced this issue Oct 26, 2024
* (HaggardFFM) allow write on segments, solves #688 #706
* (DutchmanNL) Fixed error when a device is not available Solves #698
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants