How to debug crash of image script when it's called as extension

cvp

@halloleooo perhaps find a way without passing via pil, for example, via ui.image_context?

cvp

@halloleooo if you want a join of original pictures, I think that even ui.image_context will use a lot of memory, but if you want a smaller jointed image, it could be possible to work with resized images.

cvp

@halloleooo please, try this one in the share sheet

import appex
import ui
import io
from PIL import Image

def pil2ui(imgIn):
	with io.BytesIO() as bIO:
		imgIn.save(bIO, 'PNG')
		imgOut = ui.Image.from_data(bIO.getvalue())
		del bIO
	return imgOut

ui_images = []	

# assume two images are shared
files = appex.get_attachments()
for f in files:
	pil = Image.open(f)#,mode='r')
	ui_image = pil2ui(pil)
	del pil
	w,h = ui_image.size
	wi = 400
	hi = wi*h/w
	with ui.ImageContext(wi,hi) as ctx:
		ui_image.draw(0,0,wi,hi)
		del ui_image
		ui_resize = ctx.get_image()
		ui_images.append(ui_resize)
		del ui_resize		
	
w1,h1 = ui_images[0].size
w2,h2 = ui_images[1].size
# assume images have same height
with ui.ImageContext(w1+w2,h1) as ctx:
	ui_images[0].draw(0,0,w1,h1)
	ui_images[1].draw(w1,0,w2,h2)
	ui_image_joined = ctx.get_image()
w,h = ui_image_joined.size
v = ui.ImageView()
v.frame = (0,0,400,400*h/w)
v.content_mode = ui.CONTENT_SCALE_ASPECT_FIT
v.image = ui_image_joined
v.present('')

halloleooo

@cvp Thanks for the detailed script.

One thing I am wondering about is: You get the images from the share extension as PIL images via

appex.get_attachments()

and then convert them to UI images. Isn't there a way to get them directly as UI images? I thought UI image is the iOS-native format...

cvp

@halloleooo you're right but we got some problems with images in appex mode, thus...
Else, I use this where you can set type you want

img = appex.get_image(image_type='pil')

JonB

Doesn't appex provide a get_ui_image function? That will be much lighter weight than going through PIL and back.

cvp

@JonB sure, he knows that, but in the past, we got some problems with it.

Édit: believe me, if we go through pil, it is not for the pleasure to do it.

halloleooo

@cvp So what are the problems with get_ui_image?

cvp

Correction: appex does not have a get_ui_image

cvp

We mixed with asset.get_ui_image

cvp

I have been disturbed only because the post comes from @JonB

cvp

@halloleooo doc dixit, you could try with

img = appex.get_image(image_type='ui')

#files = appex.get_attachments()
#for f in files:
files = appex.get_images(image_type='ui')
for ui_image in files:
	#pil = Image.open(f)#,mode='r')
	#ui_image = pil2ui(pil)
	#del pil

halloleooo

I think the Pyhonista docs mentions appex.get_ui_image(). Is this wrong?

cvp

@halloleooo modif just above also crashes with two big photos...

cvp

@halloleooo said:

I think the Pyhonista doco mentions appex.get_ui_image(). Is this wrong?

The internal doc, reached via help, and the source of appex.py do not mention it

cvp

@halloleooo but, as I said previously, asset.get_ui_image exists

cvp

@halloleooo print dir(appex) to confirm

cvp

@halloleooo I said"modif just above also crashes with two big photos..." but with pil it works

cvp

@halloleooo ok, now, Forget all, this works, by commenting the del ui_image
and is quicker than converting pil
even with wi=1000 with two photos of 2600x4000 pixels

# assume two images are shared
#files = appex.get_attachments()
#for f in files:
files = appex.get_images(image_type='ui')
for ui_image in files:
	#pil = Image.open(f)#,mode='r')
	#ui_image = pil2ui(pil)
	#del pil
	w,h = ui_image.size
	wi = 400
	hi = wi*h/w
	with ui.ImageContext(wi,hi) as ctx:
		ui_image.draw(0,0,wi,hi)
		#del ui_image
		ui_resize = ctx.get_image()
		ui_images.append(ui_resize)
		del ui_resize

JonB

@cvp, sorry, I did not actually try it -- per the release notes there is supposed to be appex.get_ui.image().

didn't actually look -- but