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.

Sign up for GitHub

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

Jump to bottom

Fix Italy URLs #127

Closed
augusto-herrmann opened this issue Oct 8, 2021 · 2 comments · Fixed by #130
Closed

Fix Italy URLs #127

augusto-herrmann opened this issue Oct 8, 2021 · 2 comments · Fixed by #130
Labels
Data Data sources and ingestion automation

Comments

@augusto-herrmann
Copy link
Collaborator

Many URLs in the Italy data are causing validation to fail. We should fix that so that the data passes validation.

There is a script in scripts/maintenance that should help with that.
@augusto-herrmann augusto-herrmann added the Data Data sources and ingestion automation label Oct 8, 2021
@augusto-herrmann augusto-herrmann changed the title Fix Italy data Fix Italy URLs Oct 8, 2021
@traversaro
Copy link
Contributor

The Italian data was obtained back in time from the italian official dataset. The modern version of that dataset is either https://indicepa.gov.it/ipa-dati/dataset/amministrazioni or https://indicepa.gov.it/ipa-dati/dataset/enti .

@augusto-herrmann
Copy link
Collaborator Author

We should probably open a new issue to update and automate data extraction. Thanks for the links, @traversaro.

For this issue, though, for now, the focus is in getting the data to validate. The URL http:https://ATO 2 CZ makes no sense at all. I think it should probably be just removed and left blank.

augusto-herrmann added a commit to augusto-herrmann/publicbodies that referenced this issue Oct 15, 2021
@augusto-herrmann
fix datasets#127: remove malformed url
d52a83b
@augusto-herrmann augusto-herrmann mentioned this issue Oct 15, 2021
Merged
@augusto-herrmann augusto-herrmann linked a pull request Oct 15, 2021 that will close this issue
fix #127: remove malformed url #130
Merged
augusto-herrmann added a commit that referenced this issue Oct 16, 2021
@augusto-herrmann
fix #127: remove malformed url
31be8fb
@augusto-herrmann augusto-herrmann mentioned this issue Oct 17, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Data Data sources and ingestion automation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix #127: remove malformed url augusto-herrmann/publicbodies
2 participants
@augusto-herrmann @traversaro