Skip to content

Update SPC ArcGIS products and harden the SDK - #5

Merged
Reddimus merged 12 commits into
mainfrom
fix/arcgis-layer-table
Sep 3, 2026
Merged

Reddimus merged 12 commits into
mainfrom
fix/arcgis-layer-table

Conversation

@Reddimus

@Reddimus Reddimus commented Sep 3, 2026

Copy link
Copy Markdown
Owner

Summary

  • replace scattered ArcGIS layer IDs with a tested product/day/hazard table, including watches, conditional intensity, days 4-8, and extended fire outlooks
  • normalize ArcGIS geometry and probability fields, discard explicit no-risk sentinel polygons, and refresh captured fixtures
  • update dependencies and CI with warnings-as-errors, clang-tidy, sanitizers, package-install and consumer coverage
  • tighten pagination, URL handling, process-wide curl initialization, documentation, and package metadata

Validation

  • 40/40 tests with AddressSanitizer and UndefinedBehaviorSanitizer
  • clang-format 18 and clang-tidy
  • automated C++ audit and Markdown lint
  • install plus find_package and FetchContent consumer builds
  • all 28 fixture checksums
  • live NOAA ArcGIS 11.3 metadata and all 39 feature layers

Closes the stale ArcGIS routing and SDK maintenance gaps.

Copilot AI lite review requested due to automatic review settings September 3, 2026 08:16

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Ubuntu 24.04 configured the tidy build with GCC 13, so clang-tidy 18 parsed libstdc++ 13. Clang 18 reports the older concepts feature level and libstdc++ therefore hides std::expected.

Compile and analyze the job with the matching Clang 18 and libc++ 18 packages.
@Reddimus
Reddimus merged commit a45597a into main Sep 3, 2026
7 checks passed
@Reddimus
Reddimus deleted the fix/arcgis-layer-table branch September 3, 2026 10:24
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