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

AU_SI12_NRT_R04 LANCE NRT data source retries #26

Merged
merged 8 commits into from
Feb 17, 2024

Conversation

trey-stafford
Copy link
Contributor

  • Update earthaccess library version to ~=0.8.2.
  • The code that downloads AU_SI12_NRT_R04 LANCE NRT data
    (``pm_tb_data.fetch.amsr.lance_amsr2`) now supports trying both data URLs
    provided by CMR (lance.nsstc.nasa.gov and lance.itsc.uah.edu) if the first one
    fails. Sometimes one mirror is available when the other is not.

@trey-stafford trey-stafford requested a review from sc0tts February 16, 2024 00:58
Copy link

@sc0tts sc0tts left a comment

Choose a reason for hiding this comment

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

This looks good to me.
Up to you whether to implement the "write this to a temp dir and then copy it" suggestion before merging into main

@trey-stafford
Copy link
Contributor Author

This looks good to me. Up to you whether to implement the "write this to a temp dir and then copy it" suggestion before merging into main

I ended up implementing the re-try. I actually ran into this issue today! The nasa server seems to be responding again, but it's very slow. I canceled at one point, which led to a partial download that I needed to manually cleanup. That won't be a problem anymore!

@trey-stafford trey-stafford merged commit 2124cef into main Feb 17, 2024
1 check passed
@trey-stafford trey-stafford deleted the lance-nrt-amsr2-support-retries branch February 17, 2024 00:14
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