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

For Review Only #861

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

devarishi7
Copy link

ESP32 I2S faster 3-step encoding, as well as DMX512 encoding

@@ -5,7 +5,7 @@ NeoPixel library helper functions for Methods.
Written by Michael C. Miller.

I invest time and resources providing this open source code,
please support me by donating (see https://github.com/Makuna/NeoPixelBus)
please support me by dontating (see https://github.com/Makuna/NeoPixelBus)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are introducing a typo here. 😉

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pretty sure that Typo has been there for a while. It's in the comments hardly relevant.

@@ -4,7 +4,7 @@ NeoPixel library helper functions for Esp32.
Written by Michael C. Miller.

I invest time and resources providing this open source code,
please support me by donating (see https://github.com/Makuna/NeoPixelBus)
please support me by dontating (see https://github.com/Makuna/NeoPixelBus)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are introducing a typo here. 😉

@Makuna
Copy link
Owner

Makuna commented Dec 19, 2024

(I lost my house to a tree fall and I am more than little occupied, hang in there, I will get back to this)

@edwardtfn
Copy link

Oh no!!
I wish you can recover as much as possible from the accident.
Please let us know if we can help in some way other than with donations... 😉

@devarishi7
Copy link
Author

Oh dear, i was patiently waiting, though a little curious to why it was taking so so, but not in a hurry. I have plenty to develop for now. I probably could add the 32-bit version (like for the ESP8266) since the ESP32 is only the dual 16-bit. (one could call that the mysterious Pop-endian) Hope all is well and / or soon repaired.

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