Skip to content

Baseline Removal#32

Open
CatYang3 wants to merge 2 commits into
configfrom
baseline
Open

Baseline Removal#32
CatYang3 wants to merge 2 commits into
configfrom
baseline

Conversation

@CatYang3

@CatYang3 CatYang3 commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Summary

Describe the main changes in this PR:

  • Capture the first N samples and calculate the average as the baseline.
  • Subsequent samples have the baseline value subtracted
  • Baseline is calculated after first initialization and upon reset
  • Problem: the test 'raw' data already has baseline removed at the beginning, so this functionality isn't being tested. proposed solution: record new raw data directly from IMUs

Code Quality Checklist

Before requesting review, ensure:

  • All CI/CD pipelines pass
  • Code includes docstrings and type hints
  • I added new tests where necessary
  • I ran make tree to update the README.md

Additional Notes

  • Anything else reviewers should know?
  • Attach outputs, plots, logs, or GIFs here.

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.

1 participant