Skip to content

Avoid repeated no-op admission in owned source metadata - #740

Merged
trissim merged 3 commits into
mainfrom
perf/source-metadata-fallback-20261004
Oct 5, 2026
Merged

trissim merged 3 commits into
mainfrom
perf/source-metadata-fallback-20261004

Conversation

@trissim

@trissim trissim commented Oct 5, 2026

Copy link
Copy Markdown
Collaborator

Owned metadata fallback merges repeatedly scan five component views that the same immutable owner has already admitted. Literal lookup also rescans the same ordered original/scalar namespaces. Reuse those existing owner views and derive one literal namespace, preserving first-use validation, raw mappings, original-before-scalar precedence and every fresh SourceImageIdentity.

Closes #739

Validation: 55 existing owner/provenance controls; exact metadata and fresh-identity checks on 13 captured source cohorts; four strict full-pipeline comparisons against retained qualified native outputs, including exact 3D pixels. Synced with main after #734 and passed its 44 existing spreadsheet export controls.

The captured metadata frontier saves 10.270ms (68.992→58.722ms). No end-to-end speedup is established: illumination total was 0.586→0.592s; 3D total 4.465→4.701s. The 3D processing-contract union rose about289ms versus259ms higher server execution; one pair does not isolate the cause. Both timings exclude server/registry/kernel startup. The attached receiving.json preserves the negative ordinary result. This is removal of redundant admission work, not a claim of a large pipeline gain.

Refs #739. Retain owned component and literal views while preserving fresh identities and raw mapping semantics.
Refs #739. Preserve negative ordinary comparison and strict parity evidence.
Copilot AI balanced review requested due to automatic review settings October 5, 2026 03:28

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.

@trissim
trissim merged commit e00129f into main Oct 5, 2026
3 of 37 checks passed
@trissim
trissim deleted the perf/source-metadata-fallback-20261004 branch October 5, 2026 03:28
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.

Avoid repeated no-op fallback admission on owned source metadata

2 participants