search

kelvins / municipios-brasileiros

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

Inclusão do siafi_id nos arquivos #30

Closed pedrounes1 closed 3 years ago

pedrounes1 commented 3 years ago

Conforme conversamos na issue #27 , estou fazendo o PR incluindo o siafi_id nos três arquivos com dados do municipio.

eduardo-mior commented 3 years ago

@pedrounes1 poderia já incluir no PR também o DDD e o Fuso Horário igual citado na Issue. 😄

helysousa commented 3 years ago

Código DDD dos municípios disponibilizado pela ANATEL em https://www.anatel.gov.br/dadosabertos/PDA/Codigo_Nacional/PGCN.csv

pedrounes1 commented 3 years ago

Booa @pedrounes1, valeu pela contrib 👏

Só alguns pontos:

  • Poderia retirar o espaço ao final de cada linha do .sql (antes de fechar o parênteses), por favor?
  • Tentei criar/inserir localmente em um MariaDB e tomei um erro. Alguns municípios possuem apóstrofo no nome (e.g. Alta Floresta D'Oeste) e, como o conteúdo está entre aspas simples no .sql, precisamos escapar eles usando 2 apóstrofos, como é feito aqui: https://github.com/kelvins/Municipios-Brasileiros/blob/main/sql/municipios.sql#L144
  • Se puder atualizar os exemplos no README também para incluir a nova coluna seria uma boa.

Vou fazer esses ajustes e reenviar a PR.

kelvins commented 3 years ago

@pedrounes1 mandou bem, resolvendo esses pontos que eu comentei no review, o restante está redondo por mim 👍

@eduardo-mior e @helysousa se quiserem ajudar na revisão fiquem à vontade, comentários são sempre bem vindos 😁

eduardo-mior commented 3 years ago

Código DDD dos municípios disponibilizado pela ANATEL em https://www.anatel.gov.br/dadosabertos/PDA/Codigo_Nacional/PGCN.csv

Se não me engano na época que eu fiz o cruzamento dos dados eu usei esse mesmo CSV pra adicionar o DDD na minha tabela de cidades, só eu não sei se esta atualizada, ou a cada quanto tempo isso é atualizado. O foda é que tem cidades que usam 2 DDDs ai é meio impossível ficar 100% preciso essa informação do DDD.

eduardo-mior commented 3 years ago

@kelvins O siafi_id não pode ser INT pois existem varios códigos do siafi que começam com 0 então ia ficar errado, nesse caso pode ser usado um varchar de 4 caracteres. O fuso_horario pode ser um varchar de 32 caracteres não precisa ser de 50. Além disso o siafi_id pode ser UNIQUE. Recomendo dar uma olhada no SQL que eu fiz e publiquei na issue lá: https://pastebin.com/6w2Jn8qw

pedrounes1 commented 3 years ago

Fiz as correções indicadas. Se acharem algo mais que precise ser alterado, estou à disposição.

eduardo-mior commented 3 years ago

Acredito que agora com todas as correções indicadas o PullRequest pode ser aceito. @kelvins