Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PR - Tratando repetições no bot de imagens #118

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Wilkor
Copy link

@Wilkor Wilkor commented Apr 10, 2019

Meu primeiro PR para um projeto openSource.

  • Eu criei uma logica que não haja repetições no download, para isso, precisei criar uma function chamada controlArrayImageRepeat que recebe um content como parametro e itera um array que por sua vez retorna uma array sem repetições.

Wilkor Almeida added 5 commits April 9, 2019 21:55
…que o bot não faça um downloado de uma imagem repetida, para isso, precisei criar uma function chamada controlArrayImageRepeat que recebe um content como parametro e itera um array e retorna uma array sem repetições em seus index
…que o bot não faça um downloado de uma imagem repetida, para isso, precisei criar uma function chamada controlArrayImageRepeat que recebe um content como parametro e itera um array e retorna uma array sem repetições em seus index
@leodutra
Copy link
Collaborator

A lógica de fallback de imagens para cada keyword foi removida com a mudança.

@Wilkor
Copy link
Author

Wilkor commented Apr 11, 2019

@leodutra, quando vocês diz que o fallback para as keyword foi removida, você está fazendo referência a esta linha try { if (content.downloadedImages.includes(imageUrl)) { throw new Error('Imagem já foi baixada') } pergunto isso, porquê o que eu tentei fazer é não passar nenhuma imagem repetida para a function responsavel pelo download.

@leodutra
Copy link
Collaborator

Sim, mas aí foi eliminada a lógica: sempre que uma imagem falha, por outro motivo, não ocorre retentativa de download, usando a próxima imagem possível para a sentença.

O algoritmo do Filipe baixa 2 imagens para cada sentence.
Um outro exemplo pode ser visto no fork onde estou consolidando todos os merges:
https://github.com/leodutra/video-maker/blob/core-improvements/core/image-flow.js

Se ocorrer um erro qualquer, que não seja pelo motivo de duplicação ou blacklist ou extensão inválida, esse erro não será capturado e não ocorrerá o fallback para o caso.

@Wilkor
Copy link
Author

Wilkor commented Apr 12, 2019

@leodutra, entendi, maraviha! obrigado pela explicação!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants