SirStone/SirRoboGarage
1
0
Fork 0
Code Issues 28 Pull Requests Actions Projects Releases Wiki Activity

Input cleanup: remove redundant distance from sliding window #120

New Issue
Closed
opened 2026-08-29 19:04:02 +02:00 by SirStone · 1 comment
SirStone commented 2026-08-29 19:04:02 +02:00
Owner
Copy Link
Copy Source

What to build

Remove the redundant distance entry from the sliding window's last feature group so that distance appears only once in the 91-dimensional ANN input.

The sliding window currently includes distance in its last 3-element group (indices 87-89), making it appear twice in the 91-dim input - once normalized with NormWallDistance=1000.0 inside the window, and once as normDistance(currentDist) as the 91st element.

Acceptance criteria

  • toInputVec no longer includes normalized distance in the window portion (indices 87-89)
  • Distance appears only as the sole distance input at index 90
  • InputDim constant in ann.nim changes from 91 to 90
  • Unfilled window slots (zeros) semantically represent no data rather than zero distance
  • Existing when isMainModule self-checks in data.nim still pass
  • Existing when isMainModule self-checks in ann.nim still pass

Blocked by

None - can start immediately

Parent

Closes part of #118

## What to build Remove the redundant distance entry from the sliding window's last feature group so that distance appears only once in the 91-dimensional ANN input. The sliding window currently includes distance in its last 3-element group (indices 87-89), making it appear twice in the 91-dim input - once normalized with NormWallDistance=1000.0 inside the window, and once as normDistance(currentDist) as the 91st element. ## Acceptance criteria - toInputVec no longer includes normalized distance in the window portion (indices 87-89) - Distance appears only as the sole distance input at index 90 - InputDim constant in ann.nim changes from 91 to 90 - Unfilled window slots (zeros) semantically represent no data rather than zero distance - Existing when isMainModule self-checks in data.nim still pass - Existing when isMainModule self-checks in ann.nim still pass ## Blocked by None - can start immediately ## Parent Closes part of #118
SirStone added the ready-for-agent label 2026-08-29 19:05:49 +02:00
SirStone commented 2026-08-29 20:17:45 +02:00
Author
Owner
Copy Link
Copy Source

Implemented in commit a730a9e

Summary: Removed redundant distance entry from sliding window. Distance now appears only once at index 90 in the 90-dimensional input vector (was 91). Updated InputDim constant and toInputVec() to exclude normalized distance from window portion (indices 87-89).

Files:

  • vapourbot_garage/src/vapourbot/data.nim - toInputVec signature and implementation
  • vapourbot_garage/src/vapourbot/ann.nim - InputDim constant update
Implemented in commit a730a9e **Summary:** Removed redundant distance entry from sliding window. Distance now appears only once at index 90 in the 90-dimensional input vector (was 91). Updated InputDim constant and toInputVec() to exclude normalized distance from window portion (indices 87-89). **Files:** - `vapourbot_garage/src/vapourbot/data.nim` - toInputVec signature and implementation - `vapourbot_garage/src/vapourbot/ann.nim` - InputDim constant update
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
ready-for-agent
wayfinder:grilling
Wayfinder design question to grill
 wayfinder:map
Wayfinder map issue
 wayfinder:prototype
wayfinder:research
Wayfinder research question
 wayfinder:task
Wayfinder implementation task
No Label ready-for-agent
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
SirStone
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SirStone/SirRoboGarage#120
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?