Skip to content

fix: verify adjusted loader bar metadata - #54

Merged
pjy008008 merged 1 commit into
developfrom
fix/feature-loader-adjustment-metadata
Aug 8, 2026
Merged

fix: verify adjusted loader bar metadata#54
pjy008008 merged 1 commit into
developfrom
fix/feature-loader-adjustment-metadata

Conversation

@pjy008008

Copy link
Copy Markdown
Contributor

Summary

  • require the operating loader's adjustment=all Parquet metadata for adjusted feature inputs
  • update the exact base/derived loader schema fixtures to model the production ALPACA_SIP_ALL_<resolution> feeds
  • keep the trust boundary fail-closed by proving adjustment=raw is rejected

Operational evidence

After PR #53 allowed exact version verification, the four-resolution canary advanced to Parquet validation and rejected all four sources. The staged backfill planner intentionally selects ALPACA_SIP_ALL_<resolution> feeds, while the reader incorrectly required adjustment=raw.

TDD / validation

  • red: both exact loader schema acceptance cases failed when modeled with production adjustment=all
  • 56 passed, 1 deselected for the feature-output trust-boundary suite
  • Ruff passed
  • mypy passed

Safety

This does not broaden the accepted metadata set: it replaces the wrong raw adjustment with the one production feed identity the backfill planner selects.

@pjy008008
pjy008008 merged commit 03574bc into develop Aug 8, 2026
10 checks passed
@pjy008008
pjy008008 deleted the fix/feature-loader-adjustment-metadata branch August 8, 2026 11:27
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.

1 participant