forked from RasaHQ/rasa
-
Notifications
You must be signed in to change notification settings - Fork 0
Sprint 3 Continuando os trabalhos
Iuri Severo edited this page Feb 20, 2021
·
1 revision
- 20/10/2020 a 29/10/2020
- Alexandre Miguel
- Letícia Meneses
- Lucas Dutra
- Sara Silva
- Henrique Martins
- Rômulo Souza
- Martha Dantas
Atividade | Descrição |
---|---|
Issue | #6963 - Additions to information collected by telemetry |
Pull Request | - |
O que foi feito | A pessoa que escreveu a issue pediu para adicionar dois novos dados à telemetria. Ao mostrar interesse na issue, uma pessoa do Rasa disse que talvez fosse difícil uma pessoa de fora do Rasa resolver o problema. Eu disse que uma parte da solução estava feita e pedi para que ele me explicasse melhor a outra parte. Além disso, pedi para confirmar se a minha solução parcial estava certa. Até a data de escrita desse documento, não obtive resposta. |
Problemas | Falta de comunição entre nós (Henrique e Membro do Rasa). |
Solução | Auxiliar o grupo em outras issues. |
Atividade | Descrição |
---|---|
Issue | #301 - Add out of the box emoji handling |
Pull Request | - |
O que foi feito | Estudamos sobre vetorização de emojis em texto e componentes do tensorflow, analisar a possível solução no código do rasa, obtendo rastros dos conhecimentos que obtivemos nesse tempo. |
Problemas | A issue em questão está aberta há 2 anos e carrega grande relevância ao projeto, logo muita coisa foi mudada na arquitetura do rasa, e assim, algumas soluções e dicas propostas na issue ficaram em desuso. Precisamos entender as modificações e analisar os melhores caminhos para implementar uma solução que possa lidar com emojis nos dados de treinamentos do bot. |
Solução | Em desenvolvimento |
Atividade | Descrição |
---|---|
Issue | #7000 - Improve logging messages for file format recognition during training |
Pull Request | - |
O que foi feito | Entramos em contato com a criadora da issue para mais especificações, pois a descrição feita por ela, não estava muito claro sobre a solução requerida. |
Problemas | Começamos a trabalhar na solução porém a criadora da issue nos respondeu indicando que um membro do rasa já havia sido solicitado para essa issue por estar voltada para a nova versão do RASA. |
Solução | A issue já tem responsável, portanto não faremos parte dela. |
Atividade | Descrição |
---|---|
Issue | #3153 - consider replacing os.path uses with pathlib |
Pull Request | #7118 |
O que foi feito | Foi substituído o uso da biblioteca os.path para a biblioteca pathlib no módulo de nlu. |
Problemas | Algumas das funcionalidades presentes no NLU não são testadas na execução inicial do rasa. Para que elas fossem utilizadas, deveria ser criada uma pipeline customizada passando por essas funcionalidades. Além disso, a issue é grande e foi proposto pelo mantenedor do rasa que ela seja resolvida em mais de um Pull Request. |
Solução | A forma que a dupla encontrou para contornar a situação foi testando as funcionalidades através dos testes unitários. Optamos por separar a resolução da issue por módulos para submter os Pull Requests separados. |
Atividade | Descrição |
---|---|
Issue | #6133 - Format tracker state for logging |
Pull Request | #7026 |
O que foi feito | Após review do pr foram feitas algumas mudanças do funcionamento do codigo incluido mudança de local do metodo e adição de novas informações a pedido do reviewer |
Problemas | Inicialmente tivemos dificuldade de entender onde estária as informações que foi pedido adicionar. E em segundo momento tivemos dificuldade em pensar nosso metodo com uma saída que variava com base nos dados fornecidos. |
Solução | Depois de algumas reunições entramos em consenso sobre enviar uma solução funcional mas não tão elegante. |
Atualização 2 nov | Pull Requeste aceito e merjado |