Add fl_scenarios() and fl_params() with flood scenario CSV#32
Merged
NewGraphEnvironment merged 2 commits intomainfrom Mar 19, 2026
Merged
Add fl_scenarios() and fl_params() with flood scenario CSV#32NewGraphEnvironment merged 2 commits intomainfrom
NewGraphEnvironment merged 2 commits intomainfrom
Conversation
- fl_params(): loads inst/extdata/flood_params.csv (parameter legend with units, defaults, literature sources). Accepts custom path override. - fl_scenarios(): loads inst/extdata/flood_scenarios.csv (three pre-defined flood factor scenarios: ff02 active channel, ff04 functional floodplain, ff06 valley bottom). Run column for selective execution. Accepts custom path for site-level scenarios. - flood_scenarios.csv: three scenarios with all VCA params, ecological process descriptions, and citation keys. - 11 new tests (154 total): column validation, default matching, custom path, invalid path errors. - Add tibble to Imports. Fixes #28 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- flood_params.csv: add effect column for concise parameter summaries. - Vignette: add flood factor scenarios section running fl_scenarios() at ff=2, ff=4, ff=6 with three-panel comparison plot. Replace hardcoded summary table with fl_params() rendered via knitr::kable. Add inline param comments to main fl_valley_confine() call. - fl_params.R: update @return to include effect column. - test-fl_params.R: add effect to expected columns. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
NewGraphEnvironment
added a commit
that referenced
this pull request
Apr 15, 2026
library(flooded) now prints a random quote on attach. Italic quote, grey attribution, clickable blue source hyperlink (OSC 8). Suppress via options(flooded.quote_show_source = FALSE). 157 entries across 45 voices: - 25 hip-hop: Kanye West, Royce da 5'9", Black Thought, Ab-Soul, ASAP Rocky, Danny Brown, The Weeknd, Kenny Beats, Freddie Gibbs, Madlib, Travis Scott, Erick the Architect, Zombie Juice, Meechy Darko, J. Cole, Bad Bunny, Don Toliver, Aaron Frazer, Post Malone, Mac Miller, Lil Yachty, Fre$h, Mustard, IDK, Joey Bada$$ - 20 climate: Katharine Hayhoe, Michael Mann, James Hansen, Gavin Schmidt, Kate Marvel, Kim Cobb, Johan Rockstrom, Susan Joy Hassol, Naomi Oreskes, Katharine Wilkinson, Michael Oppenheimer, Friederike Otto, Peter Kalmus, Jennifer Francis, Ben Santer, Richard Alley, Bill McKibben, David Wallace-Wells, Elizabeth Kolbert, Ayana Elizabeth Johnson Curated via the soul /quotes-enable skill. 8 parallel research agents returned 157 candidates; 2 independent fact-check agents flagged 2 drops (Ab-Soul Revolt unverified; Mac Miller paraphrase) and several URL upgrades to primary sources (GQ, W Magazine, Rolling Stone, Vulture). Infrastructure matches drift v0.2.2 / mc v0.2.6: - R/zzz.R: 12-line .onAttach, reads inst/extdata/quotes.csv, styles via cli - data-raw/quotes_build.R: source of truth tibble with full provenance - data-raw/quotes_audit.csv: generated full audit trail, Rbuildignored - inst/extdata/quotes.csv: shipped slim (quote, author, source) cli added to Imports. Version 0.2.1. Also cleaned up stale PWF from issue #28 (merged in PR #32) into planning/archive/2026-04-issue-28-fl-scenarios/. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
6 tasks
NewGraphEnvironment
added a commit
that referenced
this pull request
Apr 15, 2026
library(flooded) now prints a random quote on attach. Italic quote, grey attribution, clickable blue source hyperlink (OSC 8). Suppress via options(flooded.quote_show_source = FALSE). 157 entries across 45 voices: - 25 hip-hop: Kanye West, Royce da 5'9", Black Thought, Ab-Soul, ASAP Rocky, Danny Brown, The Weeknd, Kenny Beats, Freddie Gibbs, Madlib, Travis Scott, Erick the Architect, Zombie Juice, Meechy Darko, J. Cole, Bad Bunny, Don Toliver, Aaron Frazer, Post Malone, Mac Miller, Lil Yachty, Fre$h, Mustard, IDK, Joey Bada$$ - 20 climate: Katharine Hayhoe, Michael Mann, James Hansen, Gavin Schmidt, Kate Marvel, Kim Cobb, Johan Rockstrom, Susan Joy Hassol, Naomi Oreskes, Katharine Wilkinson, Michael Oppenheimer, Friederike Otto, Peter Kalmus, Jennifer Francis, Ben Santer, Richard Alley, Bill McKibben, David Wallace-Wells, Elizabeth Kolbert, Ayana Elizabeth Johnson Curated via the soul /quotes-enable skill. 8 parallel research agents returned 157 candidates; 2 independent fact-check agents flagged 2 drops (Ab-Soul Revolt unverified; Mac Miller paraphrase) and several URL upgrades to primary sources (GQ, W Magazine, Rolling Stone, Vulture). Infrastructure matches drift v0.2.2 / mc v0.2.6: - R/zzz.R: 12-line .onAttach, reads inst/extdata/quotes.csv, styles via cli - data-raw/quotes_build.R: source of truth tibble with full provenance - data-raw/quotes_audit.csv: generated full audit trail, Rbuildignored - inst/extdata/quotes.csv: shipped slim (quote, author, source) cli added to Imports. Version 0.2.1. Also cleaned up stale PWF from issue #28 (merged in PR #32) into planning/archive/2026-04-issue-28-fl-scenarios/. Co-authored-by: Claude Opus 4.6 (1M context) <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.
Summary
fl_params(): loads VCA parameter legend CSV with units, defaults, literature sources, effectsfl_scenarios(): loads pre-defined flood factor scenarios (ff02 active channel, ff04 functional floodplain, ff06 valley bottom) withruncolumn for selective executionflood_scenarios.csv: three scenarios with all VCA params, ecological process descriptions, citation keysflood_params.csv: addeffectcolumn for concise parameter summariesfl_params(), inline param commentstibbleto ImportsRelated issues
Test plan
🤖 Generated with Claude Code