Skip to content

Addresses #437, Access.Earth Importer#635

Merged
joshi1983 merged 2 commits intohhaccessibility:masterfrom
M4ttoF:437
Jul 20, 2018
Merged

Addresses #437, Access.Earth Importer#635
joshi1983 merged 2 commits intohhaccessibility:masterfrom
M4ttoF:437

Conversation

@M4ttoF
Copy link
Copy Markdown
Collaborator

@M4ttoF M4ttoF commented Jul 19, 2018

Run 'generate_csv' to get the csv file. 'generate_csv' must be in the same directory as cleanData.json when it is run.

Copy link
Copy Markdown
Collaborator

@joshi1983 joshi1983 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The importers/access.earth/data_backup.json file is over 13MB which could add a lot to the download size of our git repository. Can you copy it to a publicly available Google Drive link instead? Also, squash or fixup the delete of the file into the commit that originally added it.

It is nice to have the raw data available for others but making it part of our repository could nearly double the data size. For now, the main repository is roughly 18MB and it could become 30MB if you commit that file and it compresses poorly.

data=requests.get(url)
if len(data.text) < 10:
return ""
#print(data.json()[0])
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove this commented code line.

json.dump(bigList,saveData)
saveData.close()
fd.close()
if __name__ == "__main__":
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Put at least 1 empty line after the main function to break it from code after the function.

@M4ttoF
Copy link
Copy Markdown
Collaborator Author

M4ttoF commented Jul 20, 2018

@joshi1983
Changes have been made as requested. I also removed a part of the code which didnt work as intended when I updated it a bit ago.

Google Drive link to data_backup.json is here

@M4ttoF M4ttoF changed the title Adresses #437, Access.Earth Importer Addresses #437, Access.Earth Importer Jul 20, 2018
@joshi1983 joshi1983 merged commit e93b0cb into hhaccessibility:master Jul 20, 2018
@M4ttoF M4ttoF deleted the 437 branch July 20, 2018 18:47
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.

2 participants