Skip to content

Implement POSTs #36

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
relzp wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Implement POSTs #36

relzp wants to merge 3 commits into from

Conversation

@relzp
Copy link
Contributor

@relzp relzp commented Jan 9, 2026

This PR aims to implement POSTs for NUSenseIO and any subsystems we can implement a POST for (currently just the IMU).
Currently:

  • Adds a post function to NUSenseIO which will call post functions for subsystems
  • Adds a post function to IMU which checks the WHOAMI register against a hardcoded device ID.
  • Runs and checks the return value of NUSenseIO's post function in main

relzp added 3 commits January 9, 2026 16:43
@relzp
add post function to IMU
d5e09ed 
post() checks the value of the WHOAMI register against the constant IMU
device ID
@relzp
add post function to NUSenseIO
3c13fc0 
post() will run the post functions of subsystems to ensure they exist
and can be communicated with
currently just calls the IMU post function
@relzp
run NUSenseIO post() after init, before startup
c90417c
@relzp relzp requested a review from Claegtun January 9, 2026 06:14
@relzp relzp self-assigned this Jan 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Claegtun Claegtun Awaiting requested review from Claegtun

At least 1 approving review is required to merge this pull request.

Assignees

@relzp relzp

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@relzp