Skip to content

Created an example that uses displayio #2

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

Merged
merged 3 commits into from
Mar 16, 2019

Conversation

makermelissa
Copy link
Collaborator

Added a working simpletest example. Tested successfully with CircuitPython 4.0-beta4, the 2.4" TFT FeatherWing, and a Feather M4 Express. It should also work with earlier versions of CP4, but that is untested.

@makermelissa makermelissa requested a review from a team March 16, 2019 06:17
display_bus = displayio.FourWire(spi, command=tft_dc, chip_select=tft_cs)
except ValueError:
displayio.release_displays()
display_bus = displayio.FourWire(spi, command=tft_dc, chip_select=tft_cs)
Copy link

Choose a reason for hiding this comment

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

nitpick: I wonder if we could call release_displays always (maybe catch the exception if it fails) and then not have to repeat the bus initialization.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

That's a good nitpick. I changed it to do that.

@makermelissa
Copy link
Collaborator Author

It appears to be failing on Travis because sphinx can't find anything on displayio. It might be because this is extending an internal class or because CP is still in beta. Either way, my changes don't touch the file it is having issues with.

@makermelissa makermelissa merged commit c7fedd3 into adafruit:master Mar 16, 2019
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.

2 participants