Skip to content

Add scheduled workflow to detect upstream failures; set weekly cadence#1180

Merged
Robinlovelace merged 9 commits into
mainfrom
copilot/trigger-github-workflow-re-run
May 12, 2026
Merged

Add scheduled workflow to detect upstream failures; set weekly cadence#1180
Robinlovelace merged 9 commits into
mainfrom
copilot/trigger-github-workflow-re-run

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 11, 2026

Upstream dataset locations and dependencies can break silently between pushes. Adds a scheduled workflow that renders the book without deploying, surfacing failures automatically.

Changes

  • New workflow .github/workflows/scheduled.yaml
    • Triggers weekly every Monday at 06:00 UTC (0 6 * * 1) and on workflow_dispatch for manual runs
    • Runs bookdown::render_book("index.Rmd") — render only, no deploy
    • Uses same ghcr.io/geocompx/suggests:latest container as main.yaml
    • permissions: contents: read scoped explicitly
    • Inline comment on the cron expression for readability

Copilot AI and others added 3 commits May 11, 2026 12:55
Agent-Logs-Url: https://github.com/geocompx/geocompr/sessions/19f46a52-d387-423d-b4a4-554ec62aada0

Co-authored-by: Robinlovelace <1825120+Robinlovelace@users.noreply.github.com>
Agent-Logs-Url: https://github.com/geocompx/geocompr/sessions/19f46a52-d387-423d-b4a4-554ec62aada0

Co-authored-by: Robinlovelace <1825120+Robinlovelace@users.noreply.github.com>
Agent-Logs-Url: https://github.com/geocompx/geocompr/sessions/840d61e6-bed8-4133-ab8c-886a316fbbf0

Co-authored-by: Robinlovelace <1825120+Robinlovelace@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>


## ---------------------------------------------------------------------------------------------------
myurl = "/vsicurl/https://zenodo.org/record/5774954/files/clm_snow.prob_esacci.dec_p.90_500m_s0..0cm_2000..2012_v2.0.tif"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The 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.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The 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.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The 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

snow = rast(myurl)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The 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.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The 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.

@Robinlovelace
Copy link
Copy Markdown
Collaborator

@copilot fix the github actions check errors. I'm seeing this:

Error in `dyn.load()`:
! unable to load shared object '/usr/local/lib/R/site-library/V8/libs/V8.so':
  libnode.so.109: cannot open shared object file: No such file or directory
Backtrace:
    ▆
 1. └─base::loadNamespace(x)
 2.   ├─base::namespaceImportFrom(...)
 3.   │ └─base::asNamespace(ns)
 4.   └─base::loadNamespace(j <- i[[1L]], c(lib.loc, .libPaths()), versionCheck = vI[[j]])
 5.     └─base::library.dynam(lib, package, package.lib)
 6.       └─base::dyn.load(file, DLLpath = DLLpath, ...)

Quitting from 05-geometry-operations.Rmd:96-100 [05-geometry-operations-6]
Execution halted
Error in Rscript_render(f, render_args, render_meta, add1, add2) : 
  Failed to compile 05-geometry-operations.Rmd
Calls: <Anonymous> -> render_new_session -> Rscript_render
Execution halted

@Robinlovelace
Copy link
Copy Markdown
Collaborator

@copilot fix the github actions check errors. I'm seeing this:

Error in `dyn.load()`:
! unable to load shared object '/usr/local/lib/R/site-library/V8/libs/V8.so':
  libnode.so.109: cannot open shared object file: No such file or directory
Backtrace:
    ▆
 1. └─base::loadNamespace(x)
 2.   ├─base::namespaceImportFrom(...)
 3.   │ └─base::asNamespace(ns)
 4.   └─base::loadNamespace(j <- i[[1L]], c(lib.loc, .libPaths()), versionCheck = vI[[j]])
 5.     └─base::library.dynam(lib, package, package.lib)
 6.       └─base::dyn.load(file, DLLpath = DLLpath, ...)

Quitting from 05-geometry-operations.Rmd:96-100 [05-geometry-operations-6]
Execution halted
Error in Rscript_render(f, render_args, render_meta, add1, add2) : 
  Failed to compile 05-geometry-operations.Rmd
Calls: <Anonymous> -> render_new_session -> Rscript_render
Execution halted

This was a non-issue, sorted now I think.



## ---------------------------------------------------------------------------------------------------
myurl = "/vsicurl/https://zenodo.org/record/5774954/files/clm_snow.prob_esacci.dec_p.90_500m_s0..0cm_2000..2012_v2.0.tif"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The 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

snow = rast(myurl)



## ---------------------------------------------------------------------------------------------------
myurl = "/vsicurl/https://zenodo.org/record/5774954/files/clm_snow.prob_esacci.dec_p.90_500m_s0..0cm_2000..2012_v2.0.tif"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The 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.



## ---------------------------------------------------------------------------------------------------
myurl = "/vsicurl/https://zenodo.org/record/5774954/files/clm_snow.prob_esacci.dec_p.90_500m_s0..0cm_2000..2012_v2.0.tif"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The 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.

Comment thread code/chapters/08-read-write-plot.R Outdated
Copy link
Copy Markdown
Collaborator

@Robinlovelace Robinlovelace left a comment

Choose a reason for hiding this comment

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

Reverting the other change to the url as Zenodo seems to be working now.

Comment thread 08-read-write-plot.Rmd Outdated
Comment thread 08-read-write-plot.Rmd Outdated
@Robinlovelace Robinlovelace merged commit 734d547 into main May 12, 2026
3 checks passed
@Robinlovelace Robinlovelace deleted the copilot/trigger-github-workflow-re-run branch May 12, 2026 14:48
github-actions Bot pushed a commit that referenced this pull request May 12, 2026
…-github-workflow-re-run

Add scheduled workflow to detect upstream failures; set weekly cadence 734d547
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.

4 participants