search

kelvins / municipios-brasileiros

:house_with_garden: Dados relacionados aos municípios brasileiros
MIT License
1.1k stars 612 forks source link

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

Closed Gustavo-D closed 6 years ago

Gustavo-D commented 6 years ago

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)

kelvins commented 6 years ago

@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 commented 6 years ago

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 https://github.com/kelvins/Municipios-Brasileiros/issues/12