PoC: read inputs from GitHub Releases (issue #136) - #137
Draft
lbm364dl wants to merge 1 commit into
Draft
Conversation
Proof of concept for issue #136. Adds an opt-in input source that fetches <alias>.parquet from a GitHub release (default repo eduaguilera/whep_inputs, tag "poc") into a local cache, wired into the single .download_pin_paths() choke point. Default behaviour is unchanged (still the Nextcloud pins board); enable with options(whep.input_source = "github"). Verified end-to-end: build_primary_production(2015, 2016) runs entirely off the GitHub release (all 10 inputs fetched cold from the release assets), producing 91,363 rows with the expected schema and source breakdown. Not for merge as-is: patches only the parquet download path, uses a fixed tag rather than per-alias versioning, and has no tests. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Draft / proof of concept — not for merge. Demonstrates the GitHub-Releases file-store idea from #136 end to end.
What it does
Adds an opt-in input source that fetches
<alias>.parquetfrom a GitHub release into a local cache, wired into the single.download_pin_paths()choke point (R/github_inputs.R+ a 4-line hook inR/read_raw_inputs.R). Default behaviour is unchanged — still the Nextcloud pins board. Enable with:Data
The 10 inputs
build_primary_production()needs were mirrored from the current Nextcloud board to a release ineduaguilera/whep_inputs:Proof it runs
build_primary_production(start_year = 2015, end_year = 2016)withwhep.input_source = "github", cold cache:FAOSTAT_prod(90,680) +LUH2_grassland(683). Ran in ~82 s over the network.Deliberately out of scope (PoC)
build_primary_production;whep_read_file's other branches untouched).whep_inputscurrently tracks — the versioning-scheme decision from Reconsider input-data hosting: move whep_inputs off the Nextcloud pins board #136 still needs settling.download.filerather thanpiggyback(kept dependency-free for the demo).Relates to #136.
🤖 Generated with Claude Code