Skip to content

Record fire security sweep result#1278

Open
brendancol wants to merge 1 commit intomainfrom
security-sweep-state-fire-20260425
Open

Record fire security sweep result#1278
brendancol wants to merge 1 commit intomainfrom
security-sweep-state-fire-20260425

Conversation

@brendancol
Copy link
Copy Markdown
Contributor

Summary

  • Records the fire module audit in .claude/sweep-security-state.json. No code changes.
  • All seven public APIs (dnbr, rdnbr, burn_severity_class, fireline_intensity, flame_length, rate_of_spread, kbdi) are point-wise per-cell ops over input rasters. No iteration, queues, multi-channel state, random numbers, or file I/O.
  • All seven CUDA kernels have correct bounds guards. Every public function calls _validate_raster on each raster input and _validate_scalar on scalar parameters. Inputs are cast to float32 before reaching the kernels.
  • No HIGH or CRITICAL findings.

Test plan

  • State file parses as JSON
  • Entry includes last_inspected, severity_max, categories_found, and notes

Per-cell raster ops only -- no iteration, queues, or multi-channel
state. All 7 CUDA kernels have correct point-wise bounds guards.
Every public API calls _validate_raster on each input plus
.astype('f4'). No HIGH/CRITICAL findings.
@github-actions github-actions Bot added the performance PR touches performance-sensitive code label Apr 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance PR touches performance-sensitive code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant