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

[neopixel] parsing error in klipper.cfg #287

Closed
vstruk opened this issue Nov 22, 2023 · 4 comments
Closed

[neopixel] parsing error in klipper.cfg #287

vstruk opened this issue Nov 22, 2023 · 4 comments
Labels
bug Something isn't working

Comments

@vstruk
Copy link

vstruk commented Nov 22, 2023

Mobileraker can't parse a valid section in klipper cfg.

How to reproduce

Add the following lines to your klipper.cfg, restart klipper, try to connect via mobileraker

[neopixel top]
pin: z:P2.0
chain_count: 36

Observe the error:
Parsing of configfile failed: type 'String' is not a subtype of 'List<dynamic>' ...

Version Information

  • IOS
  • Mobileraker: 2.6.7
  • Klipper: v0.10.0-320-g7e654aed
  • Moonraker: v0.8.0-185-gb8ce199
@vstruk vstruk added the bug Something isn't working label Nov 22, 2023
@Clon1998
Copy link
Owner

Hey,
thanks for reporting this bug. Please provide me with the app's logs (App-Settings -> Scroll to the Bottom -> Debug Logs).
I couldn't reproduce the issue you reported with the neopixel section you provided. However, I recall that there was a modification in the output of Klipper/Moonraker while fetching neopixels. You can attempt updating Klipper to version 11.0. Once you share your app logs with me, I'll analyze the data returned and fix the problem for 10.0.

@vstruk
Copy link
Author

vstruk commented Nov 23, 2023

Please find the log attached.
mobileraker_2023-11-22T21-20-52.log

@Clon1998
Copy link
Owner

Please find the log attached. mobileraker_2023-11-22T21-20-52.log

Thanks,
I just pushed changes to resolve the issue in the future. For now, you might want to update Klipper since the next release that will include the fix is scheduled for december.

@vstruk vstruk closed this as completed Nov 26, 2023
@vstruk
Copy link
Author

vstruk commented Nov 26, 2023

Thank you!
I'll test it on release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants