Confusing unused batch_size Variable in train_vww.py
Open
Beginner friendly
Nobody has claimed this yet.
- Dominant language
- C
- Stars
- 479
- Forks
- 116
- PR merge metrics
- No merged PRs in 30d
Description
The training script defines both BATCH_SIZE and batch_size but only uses BATCH_SIZE. I suggest we remove the unused batch_size to avoid confusion.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Open benchmark/training/visual_wake_words/train_vww.py and inspect the two referenced locations around lines 19 and 33. Confirm that BATCH_SIZE is the variable used by the training script and that the separate batch_size definition is unused. Done means the confusing unused definition is removed without changing the script's behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- machine-learning
- Issue type
- Refactor
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 68/100