Skip to content

Added RAOB metadata capabilities - #26

Merged
randytpierce merged 6 commits into
mainfrom
feature_add_raob_metadata
Oct 1, 2026
Merged

randytpierce merged 6 commits into
mainfrom
feature_add_raob_metadata

Conversation

@mollybsmith-noaa

Copy link
Copy Markdown
Contributor

This PR hopefully will add the ability to do RAOB metadata without breaking anything else.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Critical query and configuration issues, plus an output-compatibility issue, remain unresolved.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 3 High severity · 1 Medium severity

Open (4)
What changed in this PR

Adds RAOB metadata support through collection-aware queries and model-level extraction.

Changes:

  • Adds METAR/RAOB collection routing.
  • Adds levels metadata and level queries.
  • Updates SQL templates, documentation, and tests.
File Summary
write-to-db.go Adds levels to model metadata.
vxMetadataUpdater.go Routes collections and assembles RAOB levels.
template-queries.go Adds level-query support and substitutions.
sqls/​getModelsWithMetadata.sql Adds collection and metadata filters.
sqls/​getModelsNoData.sql Adds collection and dataset filters.
sqls/​getDistinctLevel.sql Adds distinct RAOB level extraction.
settings.json Adds collection and RAOB settings.
README.md Documents RAOB support.
meta_update_test.go Updates configuration test structures.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread sqls/getModelsNoData.sql Outdated
Comment thread sqls/getModelsWithMetadata.sql Outdated
Comment thread vxMetadataUpdater.go
Comment thread vxMetadataUpdater.go Outdated
mollybsmith-noaa and others added 4 commits September 22, 2026 12:29
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

@randytpierce randytpierce left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

It looks ok to me, and the tests pass in my workspace. I think this improves things.

Co-authored-by: randytpierce <25647166+randytpierce@users.noreply.github.com>
Copilot AI requested a review from randytpierce October 1, 2026 16:33
@randytpierce
randytpierce merged commit 9a6e0f2 into main Oct 1, 2026
5 checks passed
@mollybsmith-noaa
mollybsmith-noaa deleted the feature_add_raob_metadata branch October 6, 2026 20:56
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.

4 participants