-
-
Notifications
You must be signed in to change notification settings - Fork 614
Add scheduled workflow to detect upstream failures; set weekly cadence #1180
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
Changes from 4 commits
3cf0885
174649f
a667dd8
6ab8b23
6924dc3
5de2b24
d8a8c7e
afec8ad
c862f69
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,18 @@ | ||
| name: Scheduled | ||
| on: | ||
| schedule: | ||
| - cron: '0 6 * * 1' # every Monday at 06:00 UTC | ||
| workflow_dispatch: | ||
| jobs: | ||
| bookdown: | ||
| name: Render-Book | ||
| runs-on: ubuntu-latest | ||
| permissions: | ||
| contents: read | ||
| container: ghcr.io/geocompx/suggests:latest | ||
| env: | ||
| GITHUB_PAT: ${{ secrets.GITHUB_TOKEN }} | ||
| steps: | ||
| - uses: actions/checkout@v2 | ||
| - name: Render Book | ||
| run: Rscript -e 'bookdown::render_book("index.Rmd")' |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -248,7 +248,7 @@ multilayer_rast = rast(multilayer_filepath) | |
|
|
||
|
|
||
| ## --------------------------------------------------------------------------------------------------- | ||
| myurl = "/vsicurl/https://zenodo.org/record/5774954/files/clm_snow.prob_esacci.dec_p.90_500m_s0..0cm_2000..2012_v2.0.tif" | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. https://zenodo.org/record/5774954/files/clm_snow.prob_esacci.dec_p.90_500m_s0..0cm_2000..2012_v2.0.tif is super slow and 140 MB. The new one is around 5 MB and much faster.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Hi @Robinlovelace -- I am not sure if this change is good. 1. Zenodo is unusually slow today -- in the past it was much faster consistently. 2. The idea in that code chunk is that we are not downloading the whole file (thus the file size does not matter) but we are just connecting to it.
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @Nowosad are you sure only part of the file gets downloaded? The next line is
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We can try reverting it to see if actions pass in any case.
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. OK I see the syntax now.. So yeah it only gets a bit I guess. |
||
| myurl = "/vsicurl/https://download.osgeo.org/geotiff/samples/usgs/o41078a5.tif" | ||
|
Robinlovelace marked this conversation as resolved.
Outdated
|
||
| snow = rast(myurl) | ||
| snow | ||
|
|
||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.