Skip to content
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

Issue 12 - População dos municípios (json e csv) #19

Closed
wants to merge 4 commits into from

Conversation

Gustavo-D
Copy link

Adicionando população estimada de todos os municípios. Fonte dados: IBGE - SIDRA (Tabela 6579 - https://sidra.ibge.gov.br/tabela/6579) (issue 12, parcial)

Adicionando população estimada de todos os municípios. Fonte dados: IBGE - SIDRA (Tabela 6579 - https://sidra.ibge.gov.br/tabela/6579)
Adicionando população residente estimada em 2017 de todos os municípios. Fonte dados: IBGE - SIDRA (Tabela 6579 - https://sidra.ibge.gov.br/tabela/6579)
@Gustavo-D Gustavo-D changed the title Municipios_Brasileiros.csv: Issue 12 - População dos municípios (json e csv) Jun 17, 2018
@kelvins
Copy link
Owner

kelvins commented Jul 14, 2018

@Gustavo-D desculpe pela demora e obrigado pela contribuição. Como são várias alterações vou ter que reservar um tempo maior para fazer o review do PR (inclusive se alguém pudesse ajudar com isso seria ótimo), mas vi que você alterou os arquivo csv e json. Acredito que seria uma boa atualizarmos também o sql para os 3 arquivos ficarem padronizados/sincronizados. Se você não puder fazer isso beleza, posso fazer quando sobrar um tempo. Valeu 👍

@kelvins
Copy link
Owner

kelvins commented Oct 21, 2018

Pensei melhor e acredito que não faz sentido incluir a população nos dados pois ela é muito variável (é estimada a cada ano) e o custo de manter isso atualizado é grande. Outro ponto é que hoje em dia os principais dados desse repo são geográficos então talvez fuja um pouco do escopo.
Ainda, nesse PR foram atualizados apenas os arquivos CSV e JSON (faltou o SQL).
Agradeço pela sua contribuição e esforço mas infelizmente, devido aos pontos apresentados a cima, estou fechando o PR.
Closes #12

@kelvins kelvins closed this Oct 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants