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

Fix for missing LEDs #19

Open
wants to merge 15 commits into
base: develop
Choose a base branch
from

Conversation

jwellman80
Copy link

@jwellman80 jwellman80 commented Dec 16, 2024

Found an issue where if you were missing a category of LEDs (Nozzle or Logo) an error would be thrown when attempting to set those LEDs. This was not allowing tool changes to happen.

Added some protection around the setting those LEDs when the nozzle_led_name, logo_led_name, or `_led_name is not defined in the tool vars.

I only have a single logo LED and no nozzle LEDs per tool. It seems this scenario can happen if you have one set of LEDs but not the other.

TheSin- and others added 13 commits November 30, 2024 11:48
This function is in the Gray area of being useful for a tool-changer, but does not directly related to the tool-changing. Thus, it has been moved to the usermods directory.

Further consideration is needed as for how to expose it as an option for the user to install.
Fixed typo/grammar in install.sh
Updated output of probe calibration to give easier to read/copy output.
@TheSin-
Copy link
Collaborator

TheSin- commented Dec 16, 2024

but silently ignoring might not be optimal either as the user will have no idea why it's not working if it should. I'll have to think about this one, there has to be a better solution.

@jwellman80
Copy link
Author

jwellman80 commented Dec 16, 2024

I think this solution is better.

Added a variable_debug_leds to the _toolchanger_led_vars. This way you can explicitly enter a debug mode and get logging output if you're having issues with your LEDs. And for people that are missing LEDs are part of their installation they won't get spammed with issues.

image

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

Successfully merging this pull request may close these issues.

4 participants