Skip to content

Claude/add scripts readme w jy pm#21

Merged
NewGraphEnvironment merged 2 commits into
mainfrom
claude/add-scripts-readme-WJyPM
Mar 22, 2026
Merged

Claude/add scripts readme w jy pm#21
NewGraphEnvironment merged 2 commits into
mainfrom
claude/add-scripts-readme-WJyPM

Conversation

@NewGraphEnvironment
Copy link
Copy Markdown
Owner

No description provided.

claude added 2 commits March 22, 2026 18:34
Plain-language documentation covering key concepts (DEM, COG, STAC,
validation caching), pipeline steps, data flow, run modes, logging,
and performance characteristics. Modelled after stac_airphoto_bc
scripts README.

Relates to NewGraphEnvironment/sred-2025-2026#8

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

https://claude.ai/code/session_01G2X4PNVsrtxWMG5ATeBoFR
Explain how acquisition dates are inferred from filenames (utm
pattern, directory path, placeholder fallback) since the source
GeoTIFFs don't carry dates internally. Update intro and STAC
definition to mention time-based querying.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

https://claude.ai/code/session_01G2X4PNVsrtxWMG5ATeBoFR
@claude
Copy link
Copy Markdown

claude Bot commented Mar 22, 2026

Documentation-only PR. Verified all referenced scripts exist and the technical details match the code.

One minor inaccuracy: The Quick Start section shows ./scripts/build_safe.sh as the first command, but build_safe.sh runs steps 1–5 (skipping detect_changes.R and s3_sync.R). The pipeline table already describes this correctly — the Quick Start command is just slightly misleading about scope. Not a blocker.

Everything else checks out:

  • Date extraction description matches date_extract_from_path() in stac_utils.py exactly (utm pattern, directory fallback, placeholder 2000-01-01, datetime_unknown=True flag)
  • Output paths in data flow diagram match actual script constants (e.g. data/urls_access_checks.csv)
  • All 19 scripts in the table exist in scripts/

LGTM.

@NewGraphEnvironment NewGraphEnvironment merged commit c63c17b into main Mar 22, 2026
2 checks passed
@NewGraphEnvironment NewGraphEnvironment deleted the claude/add-scripts-readme-WJyPM branch March 22, 2026 22:42
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.

2 participants