Skip to content

Use FVP NPU output as the conlarge_xl golden - #11

Merged
mbhssain merged 1 commit into
mainfrom
update-conlarge-xl-npu-golden
Sep 30, 2026
Merged

mbhssain merged 1 commit into
mainfrom
update-conlarge-xl-npu-golden

Conversation

@mbhssain

Copy link
Copy Markdown
Contributor

Same treatment as #10 did for efficientnet.

  • New int16 input from NumPy seed 22, shape 1x300x80.
  • Golden is the Ethos-U85 output captured on the Corstone-320 FVP through the evaluation kit's inference_compare use case (Vela 5.2.0, ethos-u85-256, AmbiqHP_HBLRAM, Dedicated_Sram_1MB, 3 MB arena). All 1371 operators run on the NPU.
  • NPU output differs from host TFLite in 204 of 76875 elements, mostly by one LSB, largest 22. Per-frame argmax is identical on all 75 frames.

Weights note: the command stream is byte-identical to main, but 12 int16 lookup-table entries in the weight blob differ by one LSB from the previous build, which came from a different host. Vela is deterministic on the machine used here. The golden was captured with these weights, so conlarge_xl_vela.npz and src/conlarge_xl_weights.txt are regenerated and must be loaded together with the new input and golden.

conlarge_xl_weights.h stays untracked, as on main.

Replace the conlarge_xl sidecars with an int16 input generated from
seed 22 and the Ethos-U85 output captured on the Corstone-320 FVP via
the evaluation kit's inference_compare use case (Vela 5.2.0,
AmbiqHP_HBLRAM, Dedicated_Sram_1MB, 3 MB arena). The NPU output differs
from host TFLite in 204 of 76875 elements; per-frame argmax is
unchanged.

The command stream is byte-identical, but 12 int16 lookup-table entries
in the weight blob move by one LSB relative to the previous build, which
came from a different host. The golden was captured with these weights,
so the npz and weights.txt are regenerated with it.
Copilot AI balanced review requested due to automatic review settings September 30, 2026 16:40

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 encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@mbhssain
mbhssain merged commit b823fde into main Sep 30, 2026
1 check passed
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