-
Notifications
You must be signed in to change notification settings - Fork 8
Subclass each display #1
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
Comments
Here is a list that I have of the displays to subclass:
|
Why do we need subclasses for each? Is it worth having a separate display breakouts repo to handle display sizing? It would be kinda like the FeatherWing repo. |
It was suggested by @ladyada in the link I have at the top. It would be to handle the displays listed above plus the ones like mentioned in this comment. |
we def need a subclass for each chip type (ST7735, ST7789...) but i dont think we need a full subclass of each display. that said, we do need to show the init values for each display because the inits do vary... |
Thanks. That's actually the approach I was taking. |
I'm making good progress on this. I got the Mini Color TFT finished and tested and plugged in the code for most of the other variants. Need to do some clean up code more, add the ST7789, test when more displays come, and add the black tabs variant. |
Finished the ST7789 class, but I'm not sure if my ST7789 display is broken since it won't work on Arduino or CP. Just a black screen with a backlight. I tried new wires and a different controller board with the same results. I'll continue playing with it, but it may have to wait until more displays come back in stock. |
I just noticed a small crack in my screen, so it must have gotten damaged in my move. :( Fortunately some more came back in stock, so now I can finish working on this. |
Received the 1.44 Inch and 1.8 Inch displays today and now have those tested and working. Also verified that a couple non-Adafruit ST7735 displays that I have are working. I just need to test the ST7789 once that comes in and this should be done. |
After thinking about it, I think the ST7789 should go in its own repo. Although it's a similar chipset, it really shares no code with the ST7735 and makes more sense to separate it. I'll have Kattni create a repo for that and I'll separate it out. |
I'm looking forward to this :) I have a bunch of the mini tfts that I've been eager to get displayio'd |
As discussed in adafruit/Adafruit_CircuitPython_FeatherWing#4 we should subclass various displays to use displayio that use 95% of the same init code so that it works in a similar manner to the Arduino. One of the main reasons for this is that in its current state, it doesn't work on the 160x80 display.
As a bonus, I would also like to bring in the ST7789 chip as a subclass since it is very similar.
The text was updated successfully, but these errors were encountered: