forked from RasaHQ/rasa
-
Notifications
You must be signed in to change notification settings - Fork 0
Sprint 1 Conhecendo o projeto
Iuri Severo edited this page Feb 20, 2021
·
1 revision
- 22/09/2020 a 06/10/2020
- Alexandre Miguel
- Letícia Meneses
- Lucas Dutra
- Sara Silva
- Henrique Martins
- Rômulo Souza
- Martha Dantas
Atividade | Descrição |
---|---|
Issue | #6571 - Add model name to the logs in CLI |
Pull Request | #6840 - Add model relative path to CLI commands logs |
O que foi feito | O PR foi revisado e aceito, porém ainda não aconteceu o merge. Adicionamos uma função para obtermos o caminho relativo da model utilizada e printamos nos logs dos comandos necessários. |
Problemas | Tivemos problemas para enteder qual problema a issue resolveria. |
Solução | Entramos em contato com a maintainer que havia proposto as melhorias. |
Atividade | Descrição |
---|---|
Issue | #3998 - Print out "Failed stories written to results/failed_stories.md" at the end of running rasa test core |
Pull Request | #6793 - Add print to confirm that the failed_test_stories.yml has been saved |
O que foi feito | Resolvemos a issue e abrimos o PR. O reviewer sugeriu algumas mudanças, que foram implementadas logo em seguida. Até a data de escrita deste documento, o PR ainda não tinha sido revisado de novo. |
Problemas | Dificuldade para encontrar onde a solução deveria ser implementada. |
Solução | Intensa busca pelos arquivos necessários (arquivo onde seria implementada a solução / arquivos com as constantes e dependências necessárias). |
Atividade | Descrição |
---|---|
Issue | #4341 - Accept pathlib model path |
Pull Request | #6826 - Cast model_path to string |
O que foi feito | Resolvemos a issue e abrimos o PR. O reviewer sugeriu algumas mudanças. Após conversas com o reviewer, decidimos implementar algumas das mudanças sugeridas, enquanto outras ficarão para outra issue. |
Problemas | Dificuldade em entender o que o reviewer queria em suas sugestões de mudança. |
Solução | Conversas com o reviewer. |
Atividade | Descrição |
---|---|
Issue | #6603 - Export data from interactive learning broken |
Pull Request |
#6879 - Export data from interactive learning broken #6914 - Export data from interactive learning broken |
O que foi feito | Resolvemos a issue e abrimos o PR. Foram solicitadas mudanças e criação de testes para funcionalidade implementada. Quando realizamos as mudanças o reviewer responsável pelo PR comentou que eles gostariam de usar as alterações na próxima release. Sendo assim, ele criou uma branch nova no rasa, deu pull da branch da nossa organização e ele mesmo implementou os testes, que estavam faltando. |
Problemas | O módulo trabalhado na issue utiliza código de diversos outros módulos, o que foi uma dificuldade no início da sprint. Além disso, pelo mesmo motivo, tivemos problemas com imports cíclicos, tanto no build quanto no linter do CI. |
Solução | No decorrer da sprint, nos familiarizamos com o código, entendendo como de fato os módulos se comunicavam. |
Atividade | Descrição |
---|---|
Issue | #6133 - Format tracker state for logging |
Pull Request | Não realizado |
O que foi feito | Foram realizadas das reuniões de parareamento. Na primeira reunião foi feito um estudo do funcionamento do log e onde ele é gerado e na segunda reunião foram discutidas possiveis soluções para a issue. Entretanto não foram implementadas nenhuma solução |
Problemas | Nosso primeiro obstacúlo foi o dificuldade de entender o logging ja existente e qual seria a solução mais interessante já que é uma issue de solução aberta (Ele é confuso e a issue busca torná -lo mais facil de entender e debugar por ele). Outro problema enfrentado foi problemas com a maquina da estudante Martha Dantas, que ficou por 3 dias da semana sem computador. Dando a esse imprevisto o estudante Alexandre optou por ater-se a outra issue prevista para ele e outro membro da equipe. |
Solução | Os problemas com equipamentos foram solucionados e foi feito um rascunho para a solução da issue que pode ser encontrado aqui |