es-ude / es-ude/OnDeviceTraining

data_loader: transform/targetTransform fields are declared but never invoked

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

Description

`dataLoader_t.transform` / `targetTransform` are stored by `initDataLoader` and documented in DataLoader.h ("used once after loading the dataset" / "after each getBatch"), but no code path ever calls them (found during #326 exploration; re-confirmed at implementation time). Either wire them up or remove the dead seams — today they read as extension points but silently do nothing.

Contributor guide

Open the contributing guide

Research direction

Start with DataLoader.h and the initDataLoader entry point, then trace where transform and targetTransform are stored and where dataset loading and getBatch are implemented. Confirm whether the documented invocation points can be reached; done means the callbacks work as documented, with coverage for both timing points, or the unused fields and documentation are removed.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
embedded-iot, machine-learning
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.