Projeto de IC by GabrielaKucuruza · Pull Request #6 · arademaker/dicionariosIBGE

GabrielaKucuruza · 2022-06-19T15:17:12Z

Atualização e adição das PNADs 2003, 2004, 2005, 2007, 2009, 2011, 2012, 2013, 2014 e 2015
Adição da POF 2017-2018
Correção de erros

Gabriela Kucuruza, João Oliveira e Tayara Causanilhas

arademaker

os arquivos .RData e .Rhistory não deveriam estar incluídos no repositório.
o arrive le.pesquisa.R foi totalmente modificado? No PR é importante descrever o que foi alterado e o motivo.
não entendi a estrutura das novas pesquisas adicionadas. Até 2012 eu tinha um arquivo .rda por PNAD. Cada .rda é um dataset e um dataset pode conter um ou mais data.frames. Na documentação, temos arquivos .Rd para o dataset e para cada data.frame. Vocês mantiveram esta estrutura? O que vcs mudaram nos arquivos .Rd já existentes? O que fizeram para as novas pesquisas? Parece que nas novas pesquisas não temos apenas dois arquivos para ler, correto?
Por que temos nomes como dic_dom_2003 e também dicdom1983?
Por que termos rot_10_pof_201 _2018.Rd e também rot9pof2008.2009.Rd? Por que não usar sempre o mesmo padrão de nomes?

arademaker · 2022-06-22T13:34:40Z

@@ -0,0 +1,3 @@
+exportPattern("^[^\\.]")


O que esta linha significa?

Esta linha já estava no código antes da modificação. O que foi inserido foram os imports necessários para corrigir os erros presentes do NOTE, conforme explicitamos no issue no nosso repositório.

De toda forma, esta linha refere-se à definição das variáveis, de forma que o pacote exporta tudo a partir do namespace que não começa com um ponto.

arademaker · 2022-06-22T13:38:07Z

@@ -0,0 +1,69 @@
+le.pesquisa <- function(dicionario, pathname.in, codigos, rotulos = NULL, tbloco = 2000, nlines = NA) {


Seria bom detalhar no PR o que foi modificado neste arquivo. Descrever de forma clara o que precisou ser mudado. Estou achando estranho TODAS as linhas terem sido modificadas. Foi isso mesmo? Ou talvez isto só tenha sido causado por vcs editarem este arquivo no Windows?

O código não foi modificado, aconteceu provavelmente por conta do Windows mesmo.

nao foi isso. parece que foi mesmo erro de como @GabrielaKucuruza iniciou o repositório dela. Tem algo muito errado ... melhor falarmos urgente.

Qual erro?

Eu estou no trabalho. Não sei a disponibilidade dos demais para uma reunião.

arademaker · 2022-06-22T13:40:55Z

-Maintainer: Erick Fonseca <erick.fonseca@fgv.br>
-Description: This package contains dictionaries for reading microdata
+Title: Dictionaries for reading microdata surveys from IBGE
+Version: 1.6


a versão deve mudar para 1.7, certo? Vcs fizeram mudanças, não pode continuar sendo 1.6

Versão alterada.

não vejo ainda... cuidado! Estou achando que vcs partiram de versão antiga do repositório... melhor conferir antes de corrigir os problemas.

A versão alterada está no repositório do nosso projeto, referente a nossa matéria.

arademaker · 2022-06-22T13:41:31Z

+Version: 1.6
+Date: 2014-08-13
+Author: Alexandre Rademaker <alexandre.rademaker@fgv.br>
+Maintainer: Alexandre Rademaker <alexandre.rademaker@fgv.br>


existe algum campo para contribuidor? Se sim, vcs poderiam colocar o nome de vocês.

Já foram alteradas a data e a versão.

arademaker · 2022-06-22T13:51:47Z

Notem que eu não posso fazer o merge porque o PR está em conflito com a versão dos arquivos no branch master. Vcs criaram o branch de trabalho a partir do master? Vejam conflitos acima... resolvam os conflitos e problemas que apontei para que eu possa aceitar o PR.

arademaker · 2022-06-22T13:53:19Z

Os issues #5 e #3 foram resolvidos no projeto?

arademaker · 2022-06-22T14:01:09Z

Talvez vcs tenham feito confusão e feito clone do repositório do Erick, que está 12 abandonado... Ele começou o pacote quando trabalhava comigo... mas apenas começou e depois eu assumi a manutenção.

arademaker · 2022-06-22T15:21:32Z

 Package: dicionariosIBGE
 Type: Package
-Title: Dictionaries for reading survey microdata from IBGE
-Version: 1.1


a versão no master do arademaker/dicionariosIBGE é 1.6. Como aqui está aparecendo a versão 1.1?

Essa foi a alteração feita hoje:

Reforço que não estou conseguindo entender como este PR foi feito. Não vou conseguir avaliar o projeto de vocês.

Boa tarde @arademaker lamento a demora, estava sem acesso a computador até o presente momento. Entendi agora que houve uma confusão na realização do PR e peço desculpas, me orientei erroneamente pela data da versão do erick, que aparecia de 8 anos atrás. Para evitar os conflitos, queria confirmar com o senhor se posso realizar um novo PR que esteja alinhado com a última versão do pacote. Fico no aguardo e agradeço desde já pela compreensão.

Obs: como @tayaracausanilhas fez atualizações no repositório do emap-ic, para evitar os problemas que tive, fiz um fork do seu repositório e então adicionei todos o nosso projeto, na minha conta pessoal. Eu realizaria o PR a partir daqui, https://github.com/GabrielaKucuruza/dicionariosIBGE,

tayaracausanilhas · 2022-06-22T16:02:47Z

Por que temos nomes como dic_dom_2003 e também dicdom1983?

Por que termos rot_10_pof_201 _2018.Rd e também rot9pof2008.2009.Rd? Por que não usar sempre o mesmo padrão de nomes?

Os nomes já foram alterados para o padrão antigo.

Erick Fonseca and others added 4 commits February 3, 2011 10:23

corrigido nome com espaco no dicPME

a571d44

email erick atualizado

01f5b61

.tar.gz e .zip atualizados

17085e2

Projeto desenvolvido na disciplina

117cdd1

arademaker requested changes Jun 22, 2022

View reviewed changes

arademaker changed the title ~~Projeto de IC - Gabriela Kucuruza, João Oliveira e Tayara Causanilhas~~ Projeto de IC Jun 22, 2022

arademaker reviewed Jun 22, 2022

View reviewed changes

		@@ -0,0 +1,69 @@
		le.pesquisa <- function(dicionario, pathname.in, codigos, rotulos = NULL, tbloco = 2000, nlines = NA) {

Conversation

GabrielaKucuruza commented Jun 19, 2022 • edited by arademaker Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

arademaker left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

arademaker commented Jun 22, 2022

Uh oh!

arademaker commented Jun 22, 2022

Uh oh!

arademaker commented Jun 22, 2022

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

tayaracausanilhas commented Jun 22, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

GabrielaKucuruza commented Jun 19, 2022 •

edited by arademaker

Loading