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

Melhorias de usabilidade + bug fixes #4

Open
wants to merge 70 commits into
base: main
Choose a base branch
from

Conversation

tamireinhorn
Copy link

@tamireinhorn tamireinhorn commented Apr 18, 2022

  • Criei .gitignore, pra limpeza dos arquivos. Ele vai LIMPAR no primeiro pull tudo que for .mp4 ou .png, mas é só você add de novo, eles só vão parar de ser tracked.
  • Resolvi um bug no qual rodar o código sem ter um arquivo chamado last.png gerava problemas de execução.
  • Melhorei usabilidade ao permitir que as variáveis de duração de slides e do vídeo venham do input do usuário. No fim, quero melhorar isso, vide as issues que criei no meu fork. (Fiz assim por que não sei se você ainda quer manter o projeto e queria ir resolvendo elas no meu ritmo anyway).
  • Agora o diretório de fotos vai ser criado caso não exista, e vai levar o nome de fotos de {arquivo}, para evitar confusões caso alguém use o programa pra diversas aulas (o que é o esperado). Além disso, isso resolve dois bugs. Um deles fazia com que não existir o diretório 'fotos' na pasta salvava todos slides na pasta principal, o que quebraria a geração de PDF. O outro é que mesmo que o usuário escrevesse a variável de diretório na mão, isso não era refletido no resto do código, ou seja, você trocaria a variável pra ser fotos de nutrição e o programa não levava isso em conta.

tamireinhorn and others added 30 commits April 14, 2022 02:08
Tem muita coisa pra mexer ainda, isso aqui é só pra minha amiga poder ser feliz.
Precisava dum branch mais limpo pra brincar.
Esse commit aqui resolve a issue #2 mas aparentemente a captura piorou. Ainda assim, melhor um programa que SEMPRE funciona. Vida é assim.
O nome da pasta pros arquivos agora vai ser sempre 'fotos de {arquivo}', e a pasta é criada caso não exista.
Improvements pro main.py
Isso aqui mata #8 porque agora atual e last são exclusivos do diretório criado para o arquivo em si.
Duração do vídeo calculada via CV
Por hora, só reorganizei os diretórios, coisa boba.
Via TDD: Agora o arquivo abre um diálogo para escolha do arquivo, e testa se é vídeo. Se não for, levanta um erro.
Permite que o file prompt do arquivo tenha uma mensagem bonitinha.
#15 e #7  andando lado a lado.
Não foi fácil fazer funcionar mas depois entendi meus erros.
I'm not doing this again, just for some cleaning.
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.

None yet

1 participant