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

epaper2in7.py constructor has a bug #14

Open
peterhinch opened this issue Dec 30, 2020 · 4 comments
Open

epaper2in7.py constructor has a bug #14

peterhinch opened this issue Dec 30, 2020 · 4 comments

Comments

@peterhinch
Copy link

peterhinch commented Dec 30, 2020

This line throws NameError: name 'LUT_BW' isn't defined. Should it read:

LUT_BW = LUT_WW

?

@peterhinch
Copy link
Author

Looking at the Waveshare code I think the above fix is correct.

I have failed to get this working: typically the code hangs forever waiting on busy. I now suspect a hardware problem with the reset line and have ordered a replacement panel. While diagnosing this I performed a detailed comparison between your code and that of Waveshare. In the process I spotted some discrepancies between your code and that of Waveshare and would appreciate your comments.

Firstly you don't set this register.

Secondly your LUT values differ, e.g. here.

Any information would be welcome.

@phuzybuny
Copy link

@peterhinch Did you manage to get this library to work?

I'm seeing the same discrepancy in the LUT values here as well.

@peterhinch
Copy link
Author

My interest in this was to develop my own library for nano-gui so I really just studied the code. I was certain I'd found a bug so I reported it. I didn't make any attempt to make this driver work as my intention was to write a driver subclassed from framebuf.

I should mention that my experience with Waveshare devices has not been good. I ended up with two units with different hardware faults. Perhaps I was unlucky. By contrast I've written drivers for many Adafruit displays: all have worked perfectly.

@phuzybuny
Copy link

@peterhinch Thank you for sharing! I see your library supports the 2.7inch e-Paper HAT. Will take a look at nano-gui instead.

Saw your notes about possible quality issues with Waveshare e-Paper displays. I haven't encountered issues with my unit but thank you for bringing it up.

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

No branches or pull requests

2 participants