-
Notifications
You must be signed in to change notification settings - Fork 18
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
Executing it in 2022 #1
Comments
I'm interested by your refactoring, can you share your code, or a PR |
Hi @nechry, |
@labrax thank, it's just for personal usage and leaning. definitively no plan for commercial use. |
Check it out at: https://github.com/labrax/SpaceWars. Don't forget to set your TTGO/display version on the TFT_eSPI file (on my Windows located at %UserProfile%\Documents\Arduino\libraries\TFT_eSPI\User_Setup.h). |
many thank @labrax I will check it soon. I will see to use platformio instead of Arduino IDE, it's more easy to configure by device the TFT_eSPI lib. I will keep you on touch. |
@labrax my refactoring code and yours are identical. But I don't get any sound during game, when I compile and download from platformio. With Arduino, yours and mine work as expected. I finally found a problem espressif/arduino-esp32#6898 |
Great to hear! |
Dear Volos,
This is a great code you have here! I am amazed by it and I had to make some changes for it to run in my system.
First, the
Tone32.h
header is not available. Now that is built-in into ESP32 libraries. Then, theBUZZER_CHANNEL
parameter is not needed anymore for the functionstone
andnoTone
, remove them.If people have not setup TFT_eSPI completely, they'll need to do the user setup: comment the line to the top (User_Setup.h), and uncomment the ttgo line (setup 25).
Thanks again
The text was updated successfully, but these errors were encountered: