es-ude / es-ude/OnDeviceTraining

test: per-tensor {1,0} BFP wire-allocator branch test + Softmax BFP-wire inference pin

Open
#418 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C
Stars
1
Forks
3
Avg merge
1d 1h
Merged PRs (30d)
8

Description

Follow-up 5 from PR #413 (BFP epic #410) — the two final-review minors.

1. The four wire allocators (`initLayerOutputs` / `initGradTensor` / `initBufferOutput` / `initBufferInput`) gained BFP arms in PR #413; the per-tensor `{1,0}` sentinel branch (template `groupSize` 0 → whole tensor as one block) has no dedicated test. Add one.
2. Softmax on a BFP wire in the inference path deserves an explicit pin (the storage-dtype guard added in PR #413 covers Softmax backward; the inference-path behavior is currently unpinned).

Contributor guide

Open the contributing guide

Research direction

Start by reading the four allocator entry points—initLayerOutputs, initGradTensor, initBufferOutput, and initBufferInput—and locate the existing BFP wire tests. Add coverage for the per-tensor {1,0} sentinel, then inspect the Softmax inference path and pin its behavior with a BFP wire. Done means both cases have explicit passing tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
machine-learning, testing
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
68/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.