Closed beatrizmilz closed 2 months ago
rafapereirabr
commented
2 months ago Oi Beatriz! Muito obrigado por todo feedback!! muito bom mesmo. Vou aceitar seu PR. Tmb vou abrir um issue sobre o seu ultimo ponto para eu lembrar de resolver isso hj mais tarde ou amanha! Muito obrigado!
beatrizmilz
commented
2 months ago Oi Beatriz! Muito obrigado por todo feedback!! muito bom mesmo. Vou aceitar seu PR. Tmb vou abrir um issue sobre o seu ultimo ponto para eu lembrar de resolver isso hj mais tarde ou amanha! Muito obrigado!
De nada! A maioria das sugestões foram correções de erros de digitação.
Oi @rafapereirabr ! O material está muito legal, parabéns!
Eu li e percebi alguns erros de digitação, alguns parágrafos que ficaram duplicados em inglês, coisas pequenas assim. Envio um PR com algumas das coisas percebidas, mas fique a vontade para usar/usar parcialmente/não usar as sugestões.
Alguns comentários específicos:
Fiz uma sugestão no arquivo
_quarto.yml: adicionei uma linha com o códigocode-link: true. Isso faz com que as funções usadas tenham um link para a documentação das mesmas. Eu acho muito útil, mas caso você acredite que não precisa, pode ignorar.Adicionei a versão em tradução do livro R4DS também. Faltam pouquíssimos capítulos para serem revisados, então acho que vale a pena colocar a referência como mais uma leitura possível.
Capítulo 4: Para o argumento columns, seria interessante indicar como conseguimos buscar as colunas possíveis e o dicionário. Isso é comentado posteriormente nos capítulos 5 e 6. Talvez adicionar um comentário com link para o capítulo 6?
https://github.com/ipeaGIT/censobr_oficina_abep_2024/blob/ce3bf384c8e0f64d4457eabd8c5c416749edd08d/4_microdados.qmd#L191-L202
Novamente, o material está incrível! Parabéns!