gnxlxnxx/BA-pid-controller
1
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

pop() probably doesn't work #1

New Issue
Open
opened 2026-08-25 18:41:40 +02:00 by gnxlxnxx · 0 comments
gnxlxnxx commented 2026-08-25 18:41:40 +02:00
Owner
Copy Link
Copy Source

This should probably decrement the end index instead of incrementing the start index

BA-pid-controller/src/ringbuffer.rs Lines 43 to 44 in 5a007851ff
self.start += 1;
self.start %= N;

This should probably decrement the end index instead of incrementing the start index https://git.gnxlxnxx.space/gnxlxnxx/BA-pid-controller/src/commit/5a007851ff79fa81962b114f92c0fc923a4b73f4/src/ringbuffer.rs#L43-L44
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
gnxlxnxx (Roman Kretschmer)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: gnxlxnxx/BA-pid-controller#1