micropython-waveshare-epaper icon indicating copy to clipboard operation
micropython-waveshare-epaper copied to clipboard

epaper2in7.py constructor has a bug

Open peterhinch opened this issue 5 years ago • 4 comments

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

LUT_BW = LUT_WW

?

peterhinch avatar Dec 30 '20 08:12 peterhinch

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.

peterhinch avatar Jan 01 '21 12:01 peterhinch

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

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

phuzybuny avatar Jul 10 '21 04:07 phuzybuny

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.

peterhinch avatar Jul 10 '21 07:07 peterhinch

@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.

phuzybuny avatar Jul 10 '21 14:07 phuzybuny