Containers for photos, with scroll, drag&drop between them

jmv38

@cvp i have finished my first version of the program
here https://gist.github.com/1b67ba85ca7bd7b23c7058216895372c
this will:

let you choose an album xxxx
load the 200 first pictures
present them in a photopicker
from which you can build photo album pages (collage)
it creates 2 new albums:
xxxx_pages : the album pages
xxxx_pages_and_photos: as it says. useful to upload the album in photoweb printing service for instance
your own photos and albums are not modified

to see how to use it tap ‘?’ button: it opens a youtube video that shows it in action.

It is designed to work in lanscape mode
let me know how it works for you.

jmv38

@cvp hello again!

I am trying to make a high definition image with draw_snapshot()
I works fine until the context width is 4000, but i get a black image when the context width is 5000 or more. I need 9000....
Any suggestion?
Thanks.

cvp

@jmv38 Sorry, I forgot to tell you, 20 days ago, I had tried your Appli.py it I don't have any album in my photos, thus I tap the X in the first screen, and your program crashes in line
inputAlbum = allAlbums[inputTitle] 😢

cvp

@jmv38 this works for me

import ui
iv = ui.ImageView()
iv.image = ui.Image.named('P1020096.JPG') # 3888 x 2592 pixels
wi,hi = iv.image.size
iv.frame = (0,0,wi,hi)
print(wi,hi)															# 3888 2592
w = 2*wi
h = 2*hi
print(w,h)																# 7776 5184
with ui.ImageContext(w,h) as ctx:
	iv.draw_snapshot()
	ui_image = ctx.get_image()
	with open('t1.jpg', 'wb') as f:
		f.write(ui_image.to_jpeg(0.9))			# 15552 x 10368

cvp

But 9000 x 6000 crashes..

jmv38

@cvp thank for the info
i check again

jmv38

@cvp i had forgotten that 4000 is really 8000 because x2
i think 8000 should be enough for prints 60 cm wide => thanks. you saved my last week of coding!

However it is strange that 15000 is ok bu not 9000...?

Concerning the use of the project without any album, i could modify this for you if you are really interested in using this code. Are you?

thanks.

cvp

@jmv38 Not interested actually, thanks. If needed, I'll create an album.

cvp

@jmv38 During my tests and my multiple crashes, I'm pretty sure I have seen a red message during one microsecond, containing the words Decompression Bombs

This is a [PIL message](Decompression Bombs) .
I didn't use PIL. Perhaps the ui.Image.save uses it...

Something like, it not sure, message disappeared immediately
/usr/lib64/python2.6/site-packages/PIL/Image.py:2261: DecompressionBombWarning: Image size (166109750 pixels) exceeds limit of 89478485 pixels, could be decompression bomb DOS attack. DecompressionBombWarning)

cvp

@jmv38 said:

However it is strange that 15000 is ok bu not 9000...?

15000 is in pixel, already multiplied by 2. The width was 7776, just < 9000

cvp

Same crash with

Image.warnings.simplefilter('error', Image.DecompressionBombWarning)
Image.MAX_IMAGE_PIXELS = 100000000000

jmv38

@cvp 5000 doesnt work for me and 5000<7000

cvp

@jmv38 and Sure that you don't have two pixels py point?
What is your device?

jmv38

@cvp i do have 2 pixels per point (ipad air)
i dont crash, it is just that the image saved is black
i checked that 4x1024 is ok and 4x1025 fails
i remember this ios limit 4096 from somwhere.

cvp

@jmv38 could you post your code, only this part, and the image you use?

jmv38

@cvp just to let you know here is my code.
It wont run because the rest of the code is missing, but it gives you the idea

def save(self):
    # make a hi resolution copy of back & images, then save it in camera roll
    xo, yo, w, h = self.page.back.frame
    c = self.page.back.background_color
    targetWidth = 4*1024
    s = targetWidth / w
    w, h = w*s, h*s
    page = ui.View( frame=(0,0,w,h), background_color=c)
    views = []
    for thumb in self.thumbs:
      x,y,w,h = thumb.frame
      x,y,w,h = (x-xo)*s, (y-yo)*s, w*s, h*s
      v = ui.View( frame=(x,y,w,h) )
      x,y,w,h = thumb.iv.frame
      x,y,w,h = x*s, y*s, w*s, h*s
      img = thumb.getImage(thumb.asset)
      iv = ui.ImageView(frame=(x,y,w,h), image=img)
      v.add_subview(iv)
      page.add_subview(v)
      views.append(v)
    # save page image in pythonista
    getTopView().add_subview(page)
    #page.bring_to_front()
    #if True: return
    path = 'temp.jpg'
    with ui.ImageContext(page.width, page.height) as ctx:
      page.draw_snapshot()                            
      ui_image = ctx.get_image()
    pil = Image.open(io.BytesIO(ui_image.to_png()))
    pil.save(path , quality=99)
    # save page image in albums
    asset = photos.create_image_asset(path)
    os.remove(path)
    getTopView().remove_subview(page)
    views = False
    console.hud_alert('saved')

looks like i must add the view to the screen to get the draw snapshot to work.

cvp

@jmv38 said:

looks like i must add the view to the screen to get the draw snapshot to work.

Not at all, I think. You've seen my little code, nothing goes to the screen

jmv38

@cvp you are correct. My pb was pbly sthg else.

cvp

@jmv38 The only 4096 limit I find for iPad Air is Max OpenGL Texture Sizes

cvp

From here

The UIImage documentation (as of iOS 10) no longer seems to mention size limitations, although if you use UIImageView with images whose dimensions are larger than the maximum texture size* supported by the device you happen to be using you do get very large memory consumption at render time.

(The memory consumption I see in Instruments seems to indicate that the entire image is put into a 32 bits per pixel buffer when the CA::Layer is rendered.)

If your app doesn't get kill by the OS due to memory usage, the UIImageView does still end up displaying the image though.

Given this, you'll still need strategies to deal with very large images.

You can check the maximum texture size using something like glGetIntegerv(GL_MAX_TEXTURE_SIZE, &maxTextureSize);. Just make sure you've set the EAGLContext current context to be something non-nil before querying OpenGL, otherwise you'll get zero.