Skip to content

WIP: make it so that doing : python download_data.py is all you need …#25

Open
agramfort wants to merge 1 commit into
ramp-kits:masterfrom
agramfort:simplify_data_download
Open

WIP: make it so that doing : python download_data.py is all you need …#25
agramfort wants to merge 1 commit into
ramp-kits:masterfrom
agramfort:simplify_data_download

Conversation

@agramfort

Copy link
Copy Markdown
Member

…from a user standpoint

wdyt @AlexandreHutton @frcaud

I would like users to be helped as much as possible. Basically with any ramp-kit you should
get started with doing:

python download_data.py
ramp-test

@tomMoral tomMoral left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks super nice, I think it would be nice to merge this before next week if we give the challenge @frcaud

Comment thread download_data.py
"There is something wrong with the data. Verify that the expected files are present."
)

print("Decrypting archive.")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
print("Decrypting archive.")
print(
"Decrypting archive.\n"
"You need a password to open the archive. Please visit "
"https://tiny.cc/ramp_stroke_data_form, fill the form and get the "
"password on the validation page."
)

Comment thread download_data.py
Comment on lines -51 to +54
wget.download(data["url"])
# wget.download(data["url"])

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not sure you want to comment this?

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