forked from RasaHQ/rasa
-
Notifications
You must be signed in to change notification settings - Fork 0
Sprint 2 Dividas Técnicas
Iuri Severo edited this page Feb 20, 2021
·
1 revision
- 05/10/2020 a 15/10/2020
- Alexandre Miguel
- Letícia Meneses
- Lucas Dutra
- Sara Silva
- Henrique Martins
- Rômulo Souza
- Martha Dantas
Atividade | Descrição |
---|---|
Issue | #6759 - Mention the State of Telemetry during rasa init |
Pull Request | #7013 |
O que foi feito | Foi criado um aviso sobre o estado habilitado ou desabilitada do Telemetria, informando os comandos para lidar ou desligar. A mensagem é mostrada quando roda o comando rasa init . |
Problemas | Não houveram problemas na realização dessa issue. O PR foi criado no repositório da equipe mas também no repositório do projeto RASA, para garantir nossa resolução. |
Solução | Foi feita uma verificação do estado da Telemetria, em que uma função para isso já existia no projeto RASA, mostrando uma mensagem indicando o a forma de ligar ou desligar a Telemetria conforme seu estado atual. |
Atividade | Descrição |
---|---|
Issue | #7009 - Module imported twice |
Pull Request | #7014 |
O que foi feito | Haviam dois imports duplicados no meio do código, sendo que foi importado no início do arquivo, então foram removidos as importações desnecessárias. |
Problemas | Não houveram problemas na realização dessa issue. |
Solução | Remoção das linhas de importações duplicadas. |
Atividade | Descrição |
---|---|
Issue | #3153 - consider replacing os.path uses with pathlib |
Pull Request | - |
O que foi feito | Devido ao grande número de alterações que a issue demanda, o mantenedor sugeriu que fossem feitos vários PRs para a issue. A dupla preferiu começar pelo módulo NLU, e já foram alterados alguns arquivos. |
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. |
Solução | A forma que a dupla encontrou para contornar a situação foi testando as funcionalidades através dos testes unitários. |
Atividade | Descrição |
---|---|
Issue | #6133 - Format tracker state for logging |
Pull Request | #7026 |
O que foi feito | Foi implementado a solução definida anteriormente pelos membros da dupla. Agora quando mostrado o modo debug o tracer dos estados possui suar ordem de ocorrência de forma objetiva e simples de ler. |
Problemas | Nenhum problema decorreu até o momento, vale lembrar que o pull request ainda será avaliado pelo meta coach para então ser submetido para o repositório do projeto RASA. |
Solução | Modelo da solução pode ser encontrado aqui |