Updates to scripts - Githubissues

spaceml-org / helio_tools

Apache License 2.0

0 stars 2 forks source link

Updates to scripts #10

Closed jejjohnson closed 8 months ago

jejjohnson commented 9 months ago

Made the scripts for downloading more command-line friendly
removed redundant warnings for the notebooks
added some utils

@nkasmanoff If possible, can you run the refactored SDO downloader and see if it works on your end? I still get the 408 errors which I still think is something server-side (or perhaps my VPN).

nkasmanoff commented 9 months ago

Will take a look and get back to you

nkasmanoff commented 9 months ago

Will take a look and get back to you

Ran0.1_data_download.ipynb, no 408 errors on my side so the issue might be connection related

nkasmanoff commented 9 months ago

@jejjohnson left some comments with minor changes that may reduce duplicate code. Imo not a blocker to merging, I can branch off of the spaceml version of this and update later on