Skip to main content
removeu 5 caracteres do conteúdo
Fonte Link
Luiz Vieira
  • 34,5mil
  • 6
  • 91
  • 160

Realmente, você está abrindo todos os arquivos para cada frame do vídeo. Ao fazer:

images = [Image.open(filename) for filename in framesMap]

Você está gerando uma lista (images) com os handles de cada arquivo retornado pelo glob da linha anterior. Por isso o erro.

Como eu já mencionei em comentário, com um vídeo de 10 minutos gravado a 30 FPS (frames por segundo), você chega facilmente a 18 mil arquivos abertos!

Altere o seu código para processar arquivo por arquivo, dessa forma você não estoura o limite de handles do seu processo.

Primeiro, altere o código da função generate_sprite_from_frames assim:

def generate_sprite_from_frames(framesPath, columns, size, output):
    framesMap = sorted(glob.glob(framesPath + "*.png"))

    masterWidth = size[0] * columns
    masterHeight = size[1] * int(math.ceil(float(len(framesMap)) / columns))

    finalImage = Image.new(mode='RGBA', size=(masterWidth, masterHeight), color=(0,0,0,0))

    for col, filename in enumerate(framesMap):
        with Image.open(filename) as img:              
             finalImage = Image.new(mode='RGBA', size=(masterWidth, masterHeight), color=(0,0,0,0))
            merge_frames_add(finalImage, columnsimage, sizecol, outputsize)

Você vai precisar fazer uma alteração também na função merge_frames para que ela possa abrir uma imagem existente (a imagem do spritesheet com todos os frames) e adicionar o frame dadoatual a ela. E pode ser tudo em memória: a função merge_frames_add simplesmente copia o frame atual em image para a imagem do spritesheet em finalImage na região de interesse (ROI) de acordo com a coluna em col.

Note que no exemplo acima eu intencionalmente mudei o nome para merge_frames_add para denotar essa alteração.

Como você vai processar dentro de um bloco width ... as ..., a cada interação o handle do arquivo é fechado quando sai desse bloco e assim você não terá mais problemas. :)

Realmente, você está abrindo todos os arquivos para cada frame do vídeo. Ao fazer:

images = [Image.open(filename) for filename in framesMap]

Você está gerando uma lista (images) com os handles de cada arquivo retornado pelo glob da linha anterior. Por isso o erro.

Como eu já mencionei em comentário, com um vídeo de 10 minutos gravado a 30 FPS (frames por segundo), você chega facilmente a 18 mil arquivos abertos!

Altere o seu código para processar arquivo por arquivo, dessa forma você não estoura o limite de handles do seu processo.

Primeiro, altere o código da função generate_sprite_from_frames assim:

def generate_sprite_from_frames(framesPath, columns, size, output):
    framesMap = sorted(glob.glob(framesPath + "*.png"))

    masterWidth = size[0] * columns
    masterHeight = size[1] * int(math.ceil(float(len(framesMap)) / columns))

    for filename in framesMap:
        with Image.open(filename) as img:              
             finalImage = Image.new(mode='RGBA', size=(masterWidth, masterHeight), color=(0,0,0,0))
            merge_frames_add(finalImage, columns, size, output)

Você vai precisar fazer uma alteração também na função merge_frames para que ela possa abrir uma imagem existente (a imagem do spritesheet com todos os frames) e adicionar o frame dado em finalImage.

Note que no exemplo acima eu intencionalmente mudei o nome para merge_frames_add para denotar essa alteração.

Como você vai processar dentro de um bloco width ... as ..., a cada interação o handle do arquivo é fechado quando sai desse bloco e assim você não terá mais problemas. :)

Realmente, você está abrindo todos os arquivos para cada frame do vídeo. Ao fazer:

images = [Image.open(filename) for filename in framesMap]

Você está gerando uma lista (images) com os handles de cada arquivo retornado pelo glob da linha anterior. Por isso o erro.

Como eu já mencionei em comentário, com um vídeo de 10 minutos gravado a 30 FPS (frames por segundo), você chega facilmente a 18 mil arquivos abertos!

Altere o seu código para processar arquivo por arquivo, dessa forma você não estoura o limite de handles do seu processo.

Primeiro, altere o código da função generate_sprite_from_frames assim:

def generate_sprite_from_frames(framesPath, columns, size, output):
    framesMap = sorted(glob.glob(framesPath + "*.png"))

    masterWidth = size[0] * columns
    masterHeight = size[1] * int(math.ceil(float(len(framesMap)) / columns))

    finalImage = Image.new(mode='RGBA', size=(masterWidth, masterHeight), color=(0,0,0,0))

    for col, filename in enumerate(framesMap):
        with Image.open(filename) as img:                              
            merge_frames_add(finalImage, image, col, size)

Você vai precisar fazer uma alteração também na função merge_frames para que ela possa abrir uma imagem existente do spritesheet com todos os frames e adicionar o frame atual a ela. E pode ser tudo em memória: a função merge_frames_add simplesmente copia o frame atual em image para a imagem do spritesheet em finalImage na região de interesse (ROI) de acordo com a coluna em col.

Note que no exemplo acima eu intencionalmente mudei o nome para merge_frames_add para denotar essa alteração.

Como você vai processar dentro de um bloco width ... as ..., a cada interação o handle do arquivo é fechado quando sai desse bloco e assim você não terá mais problemas. :)

Fonte Link
Luiz Vieira
  • 34,5mil
  • 6
  • 91
  • 160

Realmente, você está abrindo todos os arquivos para cada frame do vídeo. Ao fazer:

images = [Image.open(filename) for filename in framesMap]

Você está gerando uma lista (images) com os handles de cada arquivo retornado pelo glob da linha anterior. Por isso o erro.

Como eu já mencionei em comentário, com um vídeo de 10 minutos gravado a 30 FPS (frames por segundo), você chega facilmente a 18 mil arquivos abertos!

Altere o seu código para processar arquivo por arquivo, dessa forma você não estoura o limite de handles do seu processo.

Primeiro, altere o código da função generate_sprite_from_frames assim:

def generate_sprite_from_frames(framesPath, columns, size, output):
    framesMap = sorted(glob.glob(framesPath + "*.png"))

    masterWidth = size[0] * columns
    masterHeight = size[1] * int(math.ceil(float(len(framesMap)) / columns))

    for filename in framesMap:
        with Image.open(filename) as img:              
            finalImage = Image.new(mode='RGBA', size=(masterWidth, masterHeight), color=(0,0,0,0))
            merge_frames_add(finalImage, columns, size, output)

Você vai precisar fazer uma alteração também na função merge_frames para que ela possa abrir uma imagem existente (a imagem do spritesheet com todos os frames) e adicionar o frame dado em finalImage.

Note que no exemplo acima eu intencionalmente mudei o nome para merge_frames_add para denotar essa alteração.

Como você vai processar dentro de um bloco width ... as ..., a cada interação o handle do arquivo é fechado quando sai desse bloco e assim você não terá mais problemas. :)