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

script for downloading json #47

Merged
merged 4 commits into from
Oct 25, 2020
Merged

Conversation

tcarreira
Copy link
Collaborator

Adoro quando isto fica simples:
Screenshot_20201024_213535

@tcarreira tcarreira marked this pull request as ready for review October 25, 2020 01:05
@tcarreira
Copy link
Collaborator Author

Podem ver aqui como ele vai buscar o conteúdo estatico: http://github.carreira.pw/ate-onde-chega-cultura/mapa.html

@tcarreira tcarreira requested a review from marado October 25, 2020 01:06
@tcarreira
Copy link
Collaborator Author

resolve #28

@tcarreira tcarreira mentioned this pull request Oct 25, 2020
3 tasks
@marado marado linked an issue Oct 25, 2020 that may be closed by this pull request
3 tasks
@waldyrious
Copy link
Contributor

waldyrious commented Oct 25, 2020

Amazing work @tcarreira! Não parece ser possível ter o output em JSON simplificado tal como exemplificado em #48, apesar de o wikidata query service oferecer esse formato para download, em conjunto com um formato "JSON file (verbose)" que corresponde ao formato usado neste PR.

Em alternativa, talvez possamos fazer o download em csv ou tsv? Assim os ficheiros ficam mais pequenos, e até fica mais fácil consultá-los no github porque são mostrados como tabelas. WDYT?

@tcarreira
Copy link
Collaborator Author

A vantagem destes JSON é que são consumidos diretamente pelo JS que constrói o mapa:
http://github.carreira.pw/ate-onde-chega-cultura/mapa.html

Mas não sei a dificuldade de alterar isso no mapa.html
Se a forma de consumo dos dados for igualmente fácil, pode trocar-se facilmente.

@waldyrious
Copy link
Contributor

A vantagem destes JSON é que são consumidos diretamente pelo JS que constrói o mapa:
http://github.carreira.pw/ate-onde-chega-cultura/mapa.html

Mas não sei a dificuldade de alterar isso no mapa.html
Se a forma de consumo dos dados for igualmente fácil, pode trocar-se facilmente.

A extração dos dados não me parece particularmente casada ao formato atual (ou facilitada por ele, sequer):

const label = point.itemLabel.value;
const coords = /Point\((.*) (.*)\)/.exec(geo).slice(1, 3).map(a => parseFloat(a)).reverse();

Mas podemos tratar dessa otimização num PR à parte 👍 Como está já é uma boa melhoria!

@waldyrious
Copy link
Contributor

Update: criei o #53 para este assunto do formato dos dados brutos.

@marado marado merged commit b2849b3 into InterruptorPt:master Oct 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cache dos dados wikidata
3 participants