search

cccneto / Ibamam

Base de dados sobre multas e autuações ambientais do Instituto Brasileiro do Meio Ambiente e dos Recursos Naturais Renováveis - IBAMA.
https://cccneto.github.io/Ibamam/
Other
12 stars 5 forks source link

Sugestoes bea #6

Closed beatrizmilz closed 3 years ago

beatrizmilz commented 3 years ago

Oi Claudiano! Eu comecei a mexer no pacote. Tentei deixar descritivo o nome dos commits pra facilitar entender o que eu fiz.

Eu não cheguei a mexer em todas as funções. Mas já mexi em algumas, e nas que eu mexi, eu busquei escrever alguns testes!

Uma dúvida: não entendi a função converter. De onde vieram aqueles valores?

cccneto commented 3 years ago

Oi Beatriz! Então alguns dos valores da função converter são oriundos da equação que usamos para realizar a mudança das moedas. Outros valores eu precisarei documentar melhor, pois foi um verdadeiro parto chegar neles, especialmente para UTN e ufir (p. e.). No entanto acredito que essa função pode ser retirada do pacote. Acho que o pacote tem que ser somente pra raspar, limpar e fornecer a base pronta para o usuário.

cccneto commented 3 years ago

Acho que o pacote deveria ter apenas duas funções gerais e que seriam visíveis aos usuários. Gerar um DF ou tbl de (1) multas distribuídas e outro para (2) multas arrecadadas. Ou seja, as duas funções seriam apenas para baixar esses dados. As demais funções de tratamento e limpeza e join, estariam dentro dessas funções gerais, implicitamente, mas não deveriam estar como @export.

beatrizmilz commented 3 years ago

Obrigada por ter olhado!

Eu não criei nenhuma função nova até então, mas tinha duas funções sem documentação (a que baixa os dados brutos por estado).

Vou dar uma olhada nos exports e edito o PR!

beatrizmilz commented 3 years ago

Fiz as modificações do export! Agora apenas as funções "finais" estão exportadas

beatrizmilz commented 3 years ago

Uma dúvida: tem algumas duplas de funções que são muito similares, e pouca coisa varia.

Para facilitar a manutenção do código, o que você acha que juntar elas e ter um argumento que diferencie se é arrecadadas ou distribuidas ?

Ex: as funções dos arquivos obter_arquivos_arrecadacao e obter_arquivos_multas_distribuidas seriam unidas, e ai a funcao em vez de nao ter argumento, seria algo como obter_multas("distribuidas") , ou obter_multas("arrecadadas") obs: pensar em quais nomes podemos usar pra deixar em inglês

O que acha?

A parte boa é que tira código que repete. esses dois arquivos são muito parecidos, seria tranquilo de transformar em um só (e consequentemente menos código para manter com o tempo)

cccneto commented 3 years ago

Sua ideia é excelente! Eu já devia ter feito isso, mas não me empenhei nessa tarefa. Por favor, prossiga nessa direção. Acho que vamos ter um pacote muito mais eficiente em em termos de código.

On Mon, 17 May 2021 at 07:56 Beatriz Milz @.***> wrote:

Uma dúvida: tem algumas duplas de funções que são muito similares, e pouca coisa varia.

Para facilitar a manutenção do código, o que você acha que juntar elas e ter um argumento que diferencie se é arrecadadas ou distribuidas ?

Ex: as funções dos arquivos obter_arquivos_arrecadacao e obter_arquivos_multas_distribuidas seriam unidas, e ai a funcao em vez de nao ter argumento, seria algo como obter_multas("distribuidas") , ou obter_multas("arrecadadas")

O que acha?

A parte boa é que tira código que repete. esses dois arquivos são muito parecidos, seria tranquilo de transformar em um só (e consequentemente menos código para manter com o tempo)

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/cccneto/Ibamam/pull/6#issuecomment-842226142, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALIDJ6XJMVGQNIE3XEMHY33TODY4BANCNFSM447PF6BA .

-- Enviado do Gmail para celular