search

PiSupply / PaPiRus

Resources for PaPiRus ePaper eInk displays
https://www.pi-supply.com/product/papirus-epaper-eink-screen-hat-for-raspberry-pi/
Other
346 stars 88 forks source link

Papirusimage.write only takes 2 args #99

Closed saty9 closed 6 years ago

saty9 commented 7 years ago

the readme docs describe a 3 argument version of papirusimage.write image.write('/path/to/image', 20, (10, 10) ) however that doesn't seem to exist.

shawaj commented 7 years ago

@francesco-vannini can you shed any light on this?

Is there a 3 argument version?

GregLukosek commented 7 years ago

This is very disappointing. Why does doc describing how to use methods that aren't existing?

GregLukosek commented 7 years ago

to clarify. PapirusImage() does not contain any method except simple write that draws image full screen

shawaj commented 7 years ago

This is in a PR I believe which we will be looking at this week.

On 3 Apr 2017 9:52 am, "Greg Lukosek" notifications@github.com wrote:

to clarify. PapirusImage() does not contain any method except simple write that draws image full screen

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/PiSupply/PaPiRus/issues/99#issuecomment-291084754, or mute the thread https://github.com/notifications/unsubscribe-auth/ADNCuiugYpvCtC4wLKQhVX3gOlKtIPPjks5rsLNCgaJpZM4Mnctc .

tvoverbeek commented 6 years ago

See commit 48ca0fc on the papirus2-and-3 branch. image.py and README updated. Something similar to the not-implemented method in PapirusImage is available in PapirusComposite. Closing this issue.