Skip to content

fix(analysis): normalize memory metadata and traffic - #176

Merged
zhen8838 merged 2 commits into
tile-ai:mainfrom
zhen8838:fix/memory-metadata-normalize
Sep 22, 2026
Merged

zhen8838 merged 2 commits into
tile-ai:mainfrom
zhen8838:fix/memory-metadata-normalize

Conversation

@zhen8838

@zhen8838 zhen8838 commented Sep 21, 2026 •

Copy link
Copy Markdown
Collaborator

Why

  • Memory results were split across three metadata types and three report families, while Function traffic undercounted loop-invariant movement.

What

  • Replace the old traffic, loop-footprint, and placement attachments with MemoryMetadata on Calls and RegionMemoryMetadata on the Function under one memory report family.
  • Remove the known-invalid authored-loop footprint and report-layer cache-pressure reconstruction; uncomputed footprints are explicitly null.
  • Accumulate logical, total, and per-unit traffic independently so total and per-unit include every executed enclosing-loop occurrence.
  • Add a hand-calculated invariant-read fixture, exact report expectations, updated consumers, specs, goldens, and generated tutorials.

Contract

  • This is an intentional breaking replacement: old metadata types, aliases, and traffic/loop-footprint JSON families are removed rather than projected for compatibility. LoopRegion receives no empty memory record. A successful RegionMemoryMetadata record has required solver_status=feasible.

Risk

  • Consumers of the old Python types or JSON families must migrate to the new memory records. Correct read footprints and L2 capacity advisories remain follow-up work; footprint is null until that analysis exists.

@zhen8838
zhen8838 force-pushed the fix/memory-metadata-normalize branch from dc5ef4c to c8dd94e Compare September 22, 2026 06:06
Comment thread tests/analysis/test_analyze_by_hand.py Outdated
from tilefoundry.analysis.report import report_data


def _checked(data: dict) -> dict:

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

You don't need to test all of them, just test the key parts.

@zhen8838
zhen8838 merged commit 53c739d into tile-ai:main Sep 22, 2026
1 check passed
@zhen8838
zhen8838 deleted the fix/memory-metadata-normalize branch September 22, 2026 07:22
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