-
Notifications
You must be signed in to change notification settings - Fork 0
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
refactor: código base para Pandas e fix do bug CLI #4
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… pandas/ adicionado novas variaveis de auxilio
…de correção ainda não implementado
vianaz
reviewed
Jan 17, 2023
vianaz
reviewed
Jan 17, 2023
vianaz
reviewed
Jan 17, 2023
vianaz
reviewed
Jan 17, 2023
vianaz
reviewed
Jan 17, 2023
vianaz
reviewed
Jan 17, 2023
vianaz
reviewed
Jan 17, 2023
vianaz
reviewed
Jan 17, 2023
vianaz
reviewed
Jan 17, 2023
vianaz
reviewed
Jan 17, 2023
vianaz
requested changes
Jan 17, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alterar as coisas solicitadas, por favor
maecco
reviewed
Jan 18, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ficou muito bom!! :)))
vianaz
approved these changes
Jan 19, 2023
vianaz
changed the title
Refactor geral do código base e fix do bug CLI
refactor: código base para Pandas e fix do bug CLI
Jan 19, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Descrição
Conforme definido no SPRINT 2:
1. Identificado e solucionado a razão do erro do módulo
cli
2. Realizado um refactor utilizando a lib
pandas
1. Módulo
cli
A razão pelo erro relacionado ao módulo
cli
se dava pelo fato da lib utilizada anteriormente PyInquirer estar desatualizada e não possuir suporte para a versão 3.10 do Python.O erro era devido a uma linha de código da lib que fazia o import do
Mapping
de uma forma que não é mais suportada na versão 3.10 do Python:Anteriormente a 3.10
from collections import Mapping
Após 3.10:
from collections.abc import Mapping
Foi removida a lib PyInquirer das dependências e substituída pela lib InquirerPy que conforme a descrição do github da lib:
Ref:
https://github.com/CITGuru/PyInquirer
https://github.com/kazhala/InquirerPy
CITGuru/PyInquirer#181
2. Refactor
Conforme os critérios de aceitação definidos no JIRA, a PR contempla os seguintes itens:
O item relacionado a verificação dos dados de entrada e incorporação de mensagens de erros caso os inputs não estejam coerentes com o esperado não foi implementada ainda.
O Refactor do código realizou mudanças com enfoque na correção do simulado tipo ENEM. entretanto como a leitura dos inputs dos CSV, além de grande parte das estruturas do código, foi alterada para utilizar-se da lib
Pandas
, A implementação antiga da correção simulado tipo UFSC não funcionará.Em praticamente todas as partes da estrutura do código foi realizado alterações visando a implementação de funções e métodos que possam torná-lo mais robusto e de fácil manutenção para eventuais adições/modificações.
Foram definidas novas variáveis no arquivo
variaveis.py
de forma a padronizar e facilitar a implementação do código:As três primeiras representam os nomes das colunas dos arquivos de input, e a última representa um dicionário de auxílio para implementar a correção do simulado tipo ENEM, e com a possibilidade de expansão posteriormente para os demais tipos de simulados.
As funções alteradas/criadas na correção tipo ENEM e as relacionadas a geração da estrutura dos arquivos JSON posteriormente podem serem adaptadas e incorporadas para os demais tipos de simulado, diminuindo dessa forma a necessidade de implementação de diversas funções semelhantes para cada tipo de correção.