search

sosbrumadinho / brumadinho_location

Conjunto de ferramentas para ajudar no resgate e localização das vítimas atingidas pelo rompimento da barragem que ocorreu recentemente em Brumadinho/MG
MIT License
868 stars 129 forks source link

Corrige script csv das pessoas não encontradas #60

Closed BrunaDolavale closed 5 years ago

vaniaon commented 5 years ago

Oi, @BrunaDolavale. Qual seria a issue associada?

BrunaDolavale commented 5 years ago

Olá, vaniaon. Não havia issue aberta pra isso, mas verifiquei que havia um bug na construção do csv. Os caracteres especiais não permitiam que o mesmo fosse criado.

Abraços, Bruna Dolavale.

Em ter, 29 de jan de 2019 às 13:39, vaniaon notifications@github.com escreveu:

Oi, Bruna. Qual seria a issue associada?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/dieegom/brumadinho_location/pull/60#issuecomment-458587266, or mute the thread https://github.com/notifications/unsubscribe-auth/AbVHa6VDXfcY-eP7kyrXvnqKaHlQl3zPks5vIGs1gaJpZM4aYMMV .

disouzaleo commented 5 years ago

Oi @BrunaDolavale só não entendi porque você criou outro .CSV, não seria melhor só reescrever o data.csv ?

disouzaleo commented 5 years ago

Além disso, para atualizar a lista de pessoas que vieram a óbito faz mais sentido fazer um crawler dessa outra página aqui também.

http://brumadinho.vale.com/obitos-identificados.html

E essas alterações estão relacionadas a essa issue aqui. https://github.com/dieegom/brumadinho_location/issues/8

BrunaDolavale commented 5 years ago

Olá, @Lrodlima . Eu não alterei o propósito do código inicial. Só refatorei e corrigi o bug dos caracteres especiais. Mas tem razão, da pra melhorar o CSV e acrescentar informações. O farei...

disouzaleo commented 5 years ago

Sim @BrunaDolavale acho que fico melhor mesmo, inclusive com classe OO. Só não aceito o teu PR porque não tenho acesso de escrita, se você já conseguisse incluir os dados dos óbitos já avancaria bastante a coleta de dados, além disso. Acho que vale a pena deixar só em um CSV mesmo.

BrunaDolavale commented 5 years ago

@Lrodlima 👍 Valeu, implementarei...

caiocarrara commented 5 years ago

@BrunaDolavale gostei da alteração no crawler. Porém por que você está commitando o csv? Dado que é uma informação relativamente volátil, não seria interessente gerar o output sob demanda? Portanto não ter o arquivo csv versionado com o restante do código. O que acha?