Open
Conversation
Contributor
|
Testei e gerou o mesmo resultado nos dois códigos. Contudo, tive a impressão que o código com as implementações desse commit #158 rodou bem mais rápido. |
Dois problemas o stata não estava reconhecendo a variavel aux_ind e por isso troquei pela aux_id que já estava sendo usada, além disso o codigo se encerrava no inicio por causa de um exit prematuro.
O codigo juntava arquivos que não existiam e nem tinham sido criados
o codigo antigo tentava substituir numero por texto (replace id_ind) e o idrs chamava o idbas errado.
Baseado na metodologia antiga
Tem um "" desnecessário.
Alterando o tipo da variavel V1014
Não estavam sendo gerados corretamente.
Este commit corrige a detecção dos limites de painel (min/max V1014), reconstrói corretamente a lista de arquivos de painel, restaura a lógica de seleção de identificação da interface gráfica, e altera o comportamento dos identificadores para que sejam salvos diretamente nos arquivos de painel. Também adiciona limpeza de variáveis internas (__*), melhora a robustez na leitura de V1014 e uniformiza o fluxo entre identificação básica e avançada. Essas mudanças corrigem problemas como criação de painel inexistente (0), falhas de carregamento de arquivos (r(198)), arquivos de painel sem identificadores.
exclusão das variaveis temporarias que estavam vazando para o dta salvo
Seguindo como é feito no R, o id_ind foi transformado de string para float, assim como uma separação de id_ind (basic) para id_rs (avançado) e foram feitos outros diversos ajustes pequenos.
IDs passaram a ser calculados integralmente em formato numérico e convertidos para string apenas no final. O identificador básico (id_ind) e o avançado (id_rs) agora são distintos e não híbridos, ambos entregues no padrão "Painel_NumeroID" (exemplo "11_2203", sendo um id do painel 11).
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Currently, our PNADC panel in R outperforms the one in Stata. This doesn't make sense as they should be implementing the same methodology by Ribas & Soares (2008). This pull request completely changes the way we create the identifiers in Stata, replicating the (much simpler) process adopted in R.
We need someone to