Skip to content

Pixels <-> tensors through Sdcb.SimdOnnx.Imaging; engine 0.1.0-preview.6 - #6

Merged
sdcb merged 3 commits into
mainfrom
imaging
Oct 17, 2026
Merged

sdcb merged 3 commits into
mainfrom
imaging

Conversation

@sdcb

@sdcb sdcb commented Oct 16, 2026

Copy link
Copy Markdown
Owner
  • CPU path converts pixels to tensors and back through Sdcb.SimdOnnx.Imaging.ImageTensor (FromPixels with replicate padding, ToPixels with crop and alpha copy). The four PixelOps converters are removed.
  • Output is bit-identical: a differential test over 160 random cases (4 pixel formats × NCHW / NHWC, in and out) matched the old converters before they were deleted. Goldens pass (74/74).
  • The NCHW variants now also split rows across threads.
  • CPU / Vulkan e2e at parity with main (tools/ab-gcdr.ps1, two interleaved rounds, ±3% noise); memory unchanged.
  • Pins Sdcb.SimdOnnx 0.1.0-preview.5 (ProGet test feed), which also brings faster per-shape compiles, GPU const packing outside the GC heap and the Metal op-set additions.

🤖 Generated with Claude Code

sdcb and others added 3 commits October 16, 2026 19:40
…r (FromPixels with replicate padding, ToPixels with crop and alpha copy; bit-identical to the removed PixelOps converters on 160 random cases x 4 formats x both layouts; the NCHW variants now also split rows across threads). CPU / Vulkan e2e at parity with main. Needs the next Sdcb.SimdOnnx preview

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…s pixels; drop an unused local (review)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sdcb sdcb changed the title Pixels <-> tensors through Sdcb.SimdOnnx.Imaging; engine 0.1.0-preview.5 Pixels <-> tensors through Sdcb.SimdOnnx.Imaging; engine 0.1.0-preview.6 Oct 17, 2026
@sdcb
sdcb merged commit b84a948 into main Oct 17, 2026
72 checks 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.

1 participant