search

ACCESS-Cloud-Based-InSAR / DockerizedTopsApp

Apache License 2.0
21 stars 2 forks source link

Release v0.3.0 -- ESA CDSE cutover and timstamp fixes #157

Closed jhkennedy closed 10 months ago

review-notebook-app[bot] commented 10 months ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

cmarshak commented 10 months ago

Questions:

  1. Are the tests failing because of issues with downloading?
  2. We have a big merge in #122 - should we wait to release those together or do them separately? Thinking that should be merged before turkey day? 🤞
jhkennedy commented 10 months ago
  1. Are the tests failing because of issues with downloading?

Something threw a 504; I suspect it's transient and this round of checks to pass

  1. We have a big merge in https://github.com/ACCESS-Cloud-Based-InSAR/DockerizedTopsApp/pull/122 - should we wait to release those together or do them separately? Thinking that should be merged before turkey day? 🤞

I think we should get this out the door as soon as you're happy with these changes -- scihub orbits may shutdown at anytime so it'd be good to have already cut over.

Always better to release smaller, incremental changes faster :-D