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

Add support for epd7in3e #102

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

antoinecellerier
Copy link

@antoinecellerier antoinecellerier commented Oct 18, 2024

@robweber
Copy link
Owner

Thanks for contributing. Couple of questions:

  1. Have you tried the upstream waveshare repo in any long term capacity? Been a while since I've tried but there was a pretty critical error for most b/w EPDs where it would error out when trying to wake from sleep. I'll pull this and test on my own but just curious.
  2. Is your 7in3e implementation dependent on the upstream merge? At first glance I'm thinking no since the method calls are the same but unsure how the changed code affects actual use.

@antoinecellerier
Copy link
Author

I've not had the opportunity to test the upstream repo with other EPDs recently no. I only have the 7in5b and 7in3e displays so no opportunity to really test black and white :-/
The upstream merge isn't technically required for things to work, it just fixes the colors being all shuffled. e.g. blue shows up as red without the fix.

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