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

Add support for ThingM Blink(1) notification LED's #6156

Closed
cakrit opened this issue May 29, 2019 · 4 comments
Closed

Add support for ThingM Blink(1) notification LED's #6156

cakrit opened this issue May 29, 2019 · 4 comments
Assignees
Labels
area/health feature request New features priority/low A 'nice to have' not critical issue

Comments

@cakrit
Copy link
Contributor

cakrit commented May 29, 2019

If someone else wants to pick this up, I'd be more than happy to help with testing (I've got half-a-dozen of the USB LED modules this is supposed to work with (specifically, the mk2 revision), so I can test complex configurations), and would be happy to provide some advice on working with the CLI tool this is using, but I'm not sure that I'm the best person to push this forwards right now.

Unless you have some objection, I think the best way forwards right now would be to close this and start fresh on a new implementation, as much has changed in alarm-notify.sh since I last touched this, and what's here didn't ever consistently work to begin with.

If someone does do a new implementation, it might be worth looking instead at integrating with the Linux Kernel LED API, which includes support for the particular hardware this was designed to work with, as well as a lot of other hardware options, and thus might be more practically useful to a wider audience.

Originally posted by @Ferroin in #3956 (comment)

@cakrit
Copy link
Contributor Author

cakrit commented May 29, 2019

#3956 has a lot more details on this. If someone is interested in this, it's best to start from there.

@ilyam8 ilyam8 added the new collector Issues to create new collector modules/plugins label May 29, 2019
@cakrit
Copy link
Contributor Author

cakrit commented Feb 22, 2020

@Ferroin see if you can/want to pick this up again, otherwise please close it.

@Ferroin Ferroin added priority/low A 'nice to have' not critical issue and removed new collector Issues to create new collector modules/plugins labels Feb 24, 2020
@Ferroin
Copy link
Member

Ferroin commented Feb 24, 2020

Similar case to #6158, I do want to take care of this, but it's just too low of a priority for the moment because of all the other stuff I'm working on.

@prologic
Copy link
Contributor

I am closing this due to no response. This is a nice cool idea et all but I don't see us doing this anytime soon and it seems more like a pet project. As it stands right now the way alarm notifications works is fairly flexible that you could script up anything you wanted, including sending signals out via GPIO or IC2 and make a bunch of LEDs blink :D

@prologic prologic assigned prologic and unassigned Ferroin Jun 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/health feature request New features priority/low A 'nice to have' not critical issue
Projects
None yet
Development

No branches or pull requests

4 participants