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. :)