Skip to content

Wheel Speed#24

Merged
dnakhooda merged 124 commits into
mainfrom
wheel-speed
Mar 31, 2026
Merged

Wheel Speed#24
dnakhooda merged 124 commits into
mainfrom
wheel-speed

Conversation

@dnakhooda

@dnakhooda dnakhooda commented Mar 16, 2026

Copy link
Copy Markdown
Member

Wheel Speed

Implemented HALL Effect for Wheel Speed.

Notes

Based off of adcs branch.

@dnakhooda
dnakhooda requested a review from KSMehta11 March 22, 2026 18:42
Comment thread Core/Src/u_wheel_speed.c Outdated
Comment thread Core/Src/u_wheel_speed.c Outdated
Comment thread Core/Inc/u_wheel_speed.h

@KSMehta11 KSMehta11 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good overall, just a couple of minor changes. There are two updates in the MX file: setting the timer prescalers and enabling the input capture interrupt for both timers.

Image

Comment thread Core/Src/u_wheel_speed.c Outdated
@dnakhooda
dnakhooda requested a review from KSMehta11 March 22, 2026 21:24
Comment thread Core/Src/u_threads.c Outdated
Comment thread Core/Src/u_wheel_speed.c Outdated
Comment thread Core/Src/u_threads.c Outdated
Comment thread Core/Src/u_wheel_speed.c
Comment thread Core/Src/u_wheel_speed.c
@dnakhooda
dnakhooda requested a review from KSMehta11 March 29, 2026 19:41

@KSMehta11 KSMehta11 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, nice work. Any further tuning/changes will have to be done after testing.

@dnakhooda
dnakhooda merged commit e55e530 into main Mar 31, 2026
1 check passed
@dnakhooda
dnakhooda deleted the wheel-speed branch March 31, 2026 00:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants