Skip to content

Conversation

Rappelbox
Copy link

No description provided.

Rappelbox added 5 commits June 9, 2020 11:05
-changed to 1 second interval
-changed to make test available at any time
number of "ledsperstrip" increased for initializing run of leds
code and description cleanup
test pin check removed as its obsolete now
removed OAK Micros because of going out of business,
Integrated GermanGamingSupplies product
@MikePinball
Copy link
Contributor

I do not agree with this pull request. There is absolutely no need to remove the Oak Micros PAL board or the support for my button power-on test. I still make them and I still support my existing customers.

By all means you can add your integrated product to the readme as an alternative so both are listed.

@Rappelbox
Copy link
Author

I just wanted to make it less confusing and you publicly stated that you were not producing anymore see you own posts on vpforums.
anyways, the test button only worked if pressed before startup and had no functionality in operational mode. The serial test seems much more reliable to me. Plus, I changed the config a little so that the test will always trigger when the teensy is powered up. Makes more sense IMO, but that’s just my opinion.

@MikePinball
Copy link
Contributor

The serial test with a "T" is a good idea. Not everyone wants test the test on power up and hence the button. I do have a version where you can press the button anytime to start the test and perhaps that should eliminate the power on test. Now I have some more time I wanted to change the test to display a light chaser so it is a little easier to see what is going on.

@MikePinball
Copy link
Contributor

MikePinball commented Jun 14, 2021

BTW For completeness you should also mention one other product using these URLs:
https://shop.arnoz.com/en/led-adressables/61-teencab.html
https://shop.arnoz.com/en/accessoires/62-teensy-40-soudee-flashee.html

It also has a test button.

@arnoz26
Copy link

arnoz26 commented Jun 14, 2021

i have a BIG test buton :)

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.

3 participants