forked from RasaHQ/rasa
-
Notifications
You must be signed in to change notification settings - Fork 0
2022 2 Sprint 4
Guilherme de Morais Richter edited this page Feb 9, 2023
·
15 revisions
O time continuou buscando issues passíveis de contribuições durante a sprint com o objetivo de agregar ao projeto RASA. Tiveram contratempo com o Trabalho Individual da matéria tendo assim pouco tempo para se dedicar às contribuições.
Descrição da Tarefa | Responsável(eis) |
---|---|
Rasa OSS - #601 | Pedro Cassiano |
Rasa OSS - #574 e Rasa PR - 11983; Rasa OSS - #561 | Pedro Cassiano e Antonio Neto |
Rasa OSS - #654 | Antonio Neto |
Issue #10954 | Álvaro Leles |
Rasa OSS - #655 | Álvaro Leles e Guilherme Richter |
Rasa OSS - #650 | Guilherme Richter |
Rasa OSS - #657 | Vitor Diniz |
Rasa OSS - #339 | Vitor Diniz |
Rasa PR - 12016 e Rasa PR - 12015 | Vitor Diniz |
Rasa PR - #11934 | Ian Ferreira |
#10954 | Ítalo Vinícius, Ian Ferreira e Guilherme Richter |
Rasa OSS - #666 | Ítalo Vinícius |
Atividade | Descrição |
---|---|
Issue | Rasa OSS - #601 |
Pull Request | Não necessária |
O que foi feito | Issue fechada, pois mantenedor disse que não precisava pois se tratava de organização pessoal de código |
Problemas | Issue não necessária |
Solução | Sem solução |
Atividade | Descrição |
---|---|
Issue | Rasa OSS - #574 ✔️ |
Pull Request | Rasa PR - 11985✔️ |
O que foi feito | PR Aceito |
Problemas | Problemas com comentario em documento onde listava linear_form como opção para model_confidence no arquivo policie.mdx |
Solução | Retirar a descrição do linear_form setting e reformular texto |
Atividade | Descrição |
---|---|
Issue | Rasa OSS - #561 |
Pull Request | PR #12016 |
Problemas | Problema ao inserir uma mensagem no início de uma conversa com Rasa. Um documento de análise e contextualização está disponível nesse docs |
O que foi feito | Foi explicado o cenário do erro que o relator estava indicando, mostrando o porquê do código não ter um fluxo normal. Além disso, demonstrei uma melhor forma de adicionar uma nova mensagem durante o Interactive Learning do Rasa. Tudo foi comentado e explicado na issue. |
Solução | Por não se tratar de um bug de código, mas sim de uma dúvida do relator, foi aberto um PR apenas de documentação, para esclarecer melhor como usar o Rasa Interactive Learning |
Atividade | Descrição |
---|---|
Issue | #10954 |
Pull Request | #12015 |
O que foi feito | Revisei arquivos da documentação para corrigir erros gramaticais |
Problemas | Palavras com gramática incorreta |
Solução | Corrigir a escrita das palavras incorretas |
Atividade | Descrição |
---|---|
Issue | OSS-655 |
Pull Request | - (aguardando resposta do mantenedor) |
O que foi feito | Pesquisamos por todo o código-fonte do Rasa Core e encontramos uma enorme quantidade de variáveis de ambiente. Fizemos uma pesquisa na documentação do Rasa e encontramos uma página de variáveis de ambiente do Rasa Enterprise (https://rasa.com/docs/rasa-enterprise/installation-and-setup/configure/add-environment-variables). O Álvaro mandou essa página na issue e perguntou se essa documentação fornece o que o usuário precisa. |
Problemas | No decorrer do desenvolvimento, nos deparamos com a enorme quantidade de variáveis de ambiente que o Rasa possui, então ao encontrarmos a documentação similar já existente, preferimos perguntar aos mantenedores se ela já satisfaz. Dentre as encontradas, podemos listar: dataset, is_external, external_dataset_repository_branch, external_dataset_repository_branch, train_dir, test_dir, domain_file, config, github_actor, github_env, tag_name, image_tag, branch_name, safe_branch_name, env, home, path, log_level_libraries, log_level_matplotlib, log_level_rabbitmq, log_level_kafka, transformers_cache, dentre muitas outras. O principal problema para não termos solucionado a tempo é realmente que o padrão encontrado exige também breve explicação e valor padrão de cada. |
Solução | Variáveis coletadas, aguardando resposta para realizar PR. |
Atividade | Descrição |
---|---|
Issue | OSS-650 |
Pull Request | Não necessário |
O que foi feito | Resposta à issue demonstrando solução. |
Problemas | - |
Solução | A issue, aberta há dois anos, solicitava que a documentação do Rasa Core estivesse faltando os eventos StoryExported, Form, FormValidation, ActionExecutionRejected e uma característica do ReminderScheduled na página de rasa.core.events. Eu iria adicionar, porém, ao verificar a documentação do Rasa rasa docs, observei que tais eventos já constam listados - provavelmente postos em data posterior à criação da isse. Então sinalizei ao mantenedor que tal issue pode ser fechada pois está concluída. |
Atividade | Descrição |
---|---|
Issue | Rasa OSS - #657 |
Pull Request | - |
O que foi feito | Interação da comunidade do Rasa em relação ao treinamento de modelo no Mac M1 |
Problemas | Treinar modelos não está sendo possível utilizando mac M1: mac m1: 9284 illegal hardware instruction |
Solução | - |
Atividade | Descrição |
---|---|
Issue | Rasa OSS - #339 |
Pull Request | PR - 11982 |
O que foi feito | Foi adicionado na documentação um parágrafo sobre o UnexpecTEDIntentPolicy na seção de handling unexpected input section. Essa abordagem não constava nessa parte da documentação. |
Problemas | Não havia feito a assinatura do termo de contribuição requerido pelo Rasa. |
Solução | Foi adicionado um parágrafo sobre o UnexpecTEDIntentPolicy na seção handling unexpected input section |
Além disso, revisei os PR'S Rasa PR - 12016 e Rasa PR - 12015, que os autores são o Antônio Neto e o Álvaro. Esses são PR's de documentação, o do Antônio foi adicionado no README do projeto uma sessão de aprendizado interativo. Já o PR do Álvaro, foram feitas correções ortográficas na documentação.
Atividade | Descrição |
---|---|
Issue | |
Pull Request | Revisão PR - #11934 |
O que foi feito | Revisão de PR solicitando modificações. |
Problemas | |
Solução | O GitHub Actions solicitava que as funções estivessem devidamente documentadas com uma docstring, desta maneira estudei o Pull Request e criei as docstrings que poderiam ser adicionadas no código. |
Atividade | Descrição |
---|---|
Issue | #10954 ✔️ |
Pull Request | #11984 ✔️ |
O que foi feito | Desta vez só corrigimos algumas alterações que o Maxime solicitou, então pedimos uma nova revisão, com isso o PR foi mergeado |
Problemas | Encontramos palavras duplicadas, verificamos a sintaxe de algumas frases e verificamos se fazia sentido os textos de instruções |
Solução | Removemos as palavras redundantes e já que estávamos verificando isto em grupo e com a ajuda da internet, pudemos certificar com maior garantia se as frases estavam redundantes ou erradas. |
Atividade | Descrição |
---|---|
Issue | Rasa OSS - #666 |
Pull Request | Não foi necessário, principalmente, pois se tornou uma demanda interna |
O que foi feito | Mostrei uma possível solução para o problema do autor da issue que incrivelmente era a correta, pois inferi que a sintaxe do Yaml seria a padrão que o parser do Python faz com estes tipos de arquivos |
Problemas | O arquivo config.yaml gerado por meio da execução do Rasa estava com um erro de parse, por isso fazia um log de warning no terminal que uma das opções estavam setadas como falso, mesmo essa opção estando verdadeira |
Solução | O valor para a chave especifica do warning estava como "true" iniciando com letra minuscula, mas deveria estar como "True", infelizmente a mantenedora abriu um ticket deste problema na board privado da Rasa e fui totalmente ignorado. |