Как завершить эту функцию python для сохранения в той же папке?

Я пытаюсь написать свою первую реальную функцию python, которая делает что-то реальное. Я хочу выполнить поиск в данной папке, а затем открыть все изображения и слить их вместе, чтобы они сделали изображение диафильма. Представьте себе 5 изображений, уложенных поверх каждого из них на одном изображении.

У меня есть этот код сейчас, который должен быть довольно хорошо, но, возможно, нуждается в некоторой модификации:

import os
import Image

def filmstripOfImages():

    imgpath = '/path/here/'
    files = glob.glob(imgpath + '*.jpg')

    imgwidth = files[0].size[0]
    imgheight = files[0].size[1]
    totalheight = imgheight * len(files)

    filename = 'filmstrip.jpg'
    filmstrip_url = imgpath + filename

    # Create the new image. The background doesn't have to be white
    white = (255,255,255)
    filmtripimage = Image.new('RGB',(imgwidth, totalheight),white)  
    row = 0
    for file in files:
        img = Image.open(file)

        left = 0
        right = left + imgwidth
        upper = row*imgheight
        lower = upper + imgheight
        box = (left,upper,right,lower)
        row += 1

        filmstripimage.paste(img, box)
    try:
        filmstripimage.save(filename, 'jpg', quality=90, optimize=1)
    except:
        filmstripimage.save(miniature_filename, 'jpg', quality=90)")

Как мне изменить это, чтобы он сохранил новый файл filmstrip.jpg в том же каталоге, в котором я загрузил изображения? И у него, вероятно, есть некоторые вещи, которые отсутствуют или ошибочны, кто-нибудь понял ключ?

Ответы

Ответ 1

Это не ответ на ваш вопрос, но может быть полезно:

#!/usr/bin/env python
import Image

def makefilmstrip(images, mode='RGB', color='white'):
    """Return a combined (filmstripped, each on top of the other) image of the images.

    """
    width  = max(img.size[0] for img in images)
    height = sum(img.size[1] for img in images)

    image = Image.new(mode, (width, height), color) 

    left, upper = 0, 0
    for img in images:
        image.paste(img, (left, upper))
        upper += img.size[1]

    return image

if __name__=='__main__':
    # Here how it could be used:
    from glob import glob
    from optparse import OptionParser

    # process command-line args
    parser = OptionParser()
    parser.add_option("-o", "--output", dest="file",
                      help="write combined image to OUTPUT")

    options, filepatterns = parser.parse_args()
    outfilename = options.file

    filenames = []
    for files in map(glob, filepatterns):
        if files:
            filenames += files

    # construct image
    images = map(Image.open, filenames)    
    img = makefilmstrip(images)
    img.save(outfilename)

Пример:

$ python filmstrip.py -o output.jpg *.jpg

Ответ 2

Я думаю, если вы измените раздел try на это:

filmstripimage.save(filmstrip_url, 'jpg', quality=90, optimize=1)

Ответ 3

В случае, если вы не шутите, есть несколько проблем с вашим script, например. glob.glob() возвращает список имен файлов (строковые объекты, а не объекты изображения), поэтому files[0].size[0] не будет работать.

Ответ 4

как отметил Дж. Ф. Себастьян, glob не возвращает объекты изображения... но также:

Как сейчас, script предполагает, что изображения в папке имеют одинаковый размер и форму. Это не всегда безопасное предположение.

Итак, по обоим причинам вам нужно будет открыть изображения, прежде чем вы сможете определить их размер. После его открытия вы должны установить ширину и масштабировать изображения до этой ширины, чтобы не было пустого пространства.

Кроме того, вы не установили miniature_filename в любом месте script.