-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
adding simpletest for esp32s2 tft #8457
base: main
Are you sure you want to change the base?
Conversation
This should be suitable for forward and reverse display versions of the ESP32-S2 and ESP32-S3 TFT. We may want to list the targeted boards explicitly somewhere. But if it lives in the board directory, then we'll have duplicates or aliases? |
I think @dglaude was interested in this too. |
I had imagined there being duplicates in cases where the hardware is similar enough to use the exact same script. I don't know how many cases like that there are, I'm open to other ideas on how handle them. Is it possible in a git repo to check in symbolic links? so this file could exist in one place and then be linked to elsewhere to have a sort of "shadow copy" exist there? |
I did not anticipate that the same simpletest would fit many board, and that simpletest proposal would start to appear so fast. |
I did hope that this one could be a good starting point as inspiration for others. I'm open to any feedback on it. I do think the built-in hardware on a given device will factor in heavily to it's simpletest so there will be some parts that aren't as re-usable between different devices. Some kind of automated generation would be neat, but I'm not opposed to sourcing some of it it from human activity. It gives us the opportunity for some new "good first issue" type things that people with less experience can tackle to get involved with the project or as contributions for hacktoberfest. |
I think you can but I'd rather just have one script per board. That'd ensure that editing one doesn't impact another. |
I'm converting this to draft because I think the best alternatives are still being discussed. |
Any plans to get back to this @FoamyGuy ? |
I missed this ping back in July. I'd be happy to get back to this, I do still think this "device simpletest" scripts can be quite useful personally. I'm open to any feedback regarding the test script in the PR, or anything meta like where it's stored etc... It sounds like from the comments above we prefer to have 1 per device so that changes can only impact a single one, that is good with me. I'm happy to make any changes that would necessary to get this one into an acceptable shape for merging. |
As discussed in the weeds a few weeks back, a board specific simpletest script.
I did the Feather ESP32-S2 TFT and made the script illustrate the usage of:
I'm interested if anyone has ideas about what else should (or shouldn't) be included in the simpletest script. I wanted to try to feature the built-in hardware as best as possible without making the script overly complex.
One idea I had but didn't do was having it do an I2C bus scan and putting any addresses it finds on the display as well. If that sounds like a good addition let me know and I can add it with another commit.
I believe this is the first one of these, so there may be actions adjustments needed if there are tools that aren't expecting to find the .py files in this directory, or perhaps pylint / black configuration may need to be altered.